Skip to content
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

Make bookmarks panel grow as screen permits #3316

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jotaemepereira
Copy link
Collaborator

@jotaemepereira jotaemepereira commented Sep 19, 2024

Task/Issue URL: https://app.asana.com/0/1204006570077678/1208262956542590/f
Tech Design URL:
CC:

Description

Acceptance criteria

AC#1
Given a user with a lot of bookmarks
When the user opens the bookmarks panel
Then, the popover will try to take as much available space either to the top or the bottom of the bookmarks popover button

AC#1
Given a user with some bookmarks
When the user opens the bookmarks panel
Then, the popover will be at least 420 wide and 500 taller.

Demo

⚠️ Ignore beach ball of death. The problem is related to having a lot of bookmarks.

make-the-bookmarks-panel-grow-as-the-screen-permits.mov

Steps to test this PR:

  1. You can load 3k bookmarks from the asana ticket
  2. Move the window around and test whether the popover takes the space either to the top or bottom

Definition of Done:

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant