-
Notifications
You must be signed in to change notification settings - Fork 83
Add persistent inspector sidebar with modular widget system #2543
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
Various dependencies have been updated to their latest versions in package-lock.json, including @InQuirer, @radix-ui, @types/node, @WordPress, and axios packages. This ensures compatibility, security, and access to new features and bug fixes.
Resolved conflicts by: - Using simpler loading/empty state logic in popular-tags component - Adding Page wrapper with title/subtitle to feed stage - Imported Page component in feed stage
- Updated build files to include Page component changes - Applied inspector sidebar max-width styling
- Updated build files to include Page component changes - Applied inspector sidebar max-width styling
Replaces the unminified JavaScript output for feed-inspector.js and feed-stage.js with minified versions, reducing file size and improving load performance. No functional code changes; only the build output is affected.
0547bf7 to
0a05321
Compare
Introduces a popular tags section in the sidebar for the social web interface, allowing users to filter posts by frequently used tags. This enhances content discovery and navigation by tag, in addition to the existing object type filters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
|
Yeah, I'm conflicted about it. Pro:
Con:
I think it's a fine experiment but doesn't feel like a slam dunk. |
thanks
If it is about colors, then let's change it. The PR is mainly meant to check a permanent three column layout and make use of the empty default space.
I wouldn't see that as an argument against that UI.
and
See it more as a proof of concept. I wanted to show a working prototype, but I am aware that there is still a lot to improve/align if we want to go that path.
The whole idea of this design was to improve the three column view and to use the free space for something useful. If the individual posts should open in main space, the right sidebar would unnecessarily limit the space we have. What I tried to say: I think we can discuss this in a different context, but a right sidebar is not the main intention of this PR. |
Proposed changes:
Implements a persistent inspector sidebar for the feed reader that is always visible, similar to Mastodon's UI design. The sidebar displays contextual information and automatically switches between a default view (showing trending tags) and post detail view when items are selected.
Key Changes:
New Components:
Layout Improvements:
Architecture:
Visual Refinements:
Other information:
Testing instructions:
Changelog entry
Changelog Entry Details
Significance
Type
Message
Add persistent inspector sidebar with modular widget system to Social Web feed reader