-
Notifications
You must be signed in to change notification settings - Fork 7
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
Bookmark folder popup position not always ideal #1155
Comments
I'll add that popups open beside the selected item, and if it's a folder near the bottom of the screen - there's no space to display its contents. |
We can now access Bookmarks using side panel on any page so this shouldn't be an issue any more. |
@guest271314 it's more about having immediate access to the bookmarks on the new tab page and hiding bookmarks elsewhere. In the same vane the native bookmarks bar is available any time, however, there's no way to show and hide it programmatically. Additionally, I've come to really enjoy this extension's bookmark bar implementation which shows folder on hover. It's really fast and convenient to access bookmarks. That said, the UI still needs a lot of work. I've been following along with progress on the CSS anchor positioning API and the popover API as potential solutions to reduce the amount of code required compared to all the positioning JS needed. Refactoring the UI and combustibility are things I'd definitely like to get to soon. |
We can programmatically open the side panel, and provide any content we want there, besides the default list that includes the Bookmarks, here on the extension defined new tab page that I use We can certainly create an element that is displayed the same window positioning as the native Chromium UI bookmarks panel. Maybe I'm not getting what you are describing. |
Now that #1151 is merged we have bookmark folder popup position that is "good enough" for general use but the experience is not ideal in all scenarios.
For context, "top level" means a folder is on the bookmark bar itself and not nested inside another folder.
Folder popup position scenarios
The current popup position logic from #1151 covers everything other than opening subsequent nested folder popups to the left (and then to the right if overflowing the left edge of the viewport).
Other issues
The text was updated successfully, but these errors were encountered: