Skip to content

Conversation

@jonasfj
Copy link
Member

@jonasfj jonasfj commented Oct 1, 2024

  • On standalone pages the banner will sit in page and be hidden.
  • On package page (and landing page) the banner will float over content with position absolute, as these pages have sufficient margin to allow for this.

In both cases content will not jump when the banner is dismissed, or when the banner is loaded.

 * On standalone pages the banner will sit in page and be hidden.
 * On package page (and landing page) the banner will float over
   content with position absolute, as these pages have sufficient
   margin to allow for this.

In both cases content will not jump when the banner is dismissed,
or when the banner is loaded.
@jonasfj jonasfj requested a review from isoos October 1, 2024 14:48
Copy link
Collaborator

@isoos isoos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you push this to staging?

@jonasfj
Copy link
Member Author

jonasfj commented Oct 2, 2024

Pushed to staging-b

@jonasfj jonasfj force-pushed the improved-banner-css branch from 01d3d31 to ff418d7 Compare October 3, 2024 09:33

.dismisser {
float: right;
position: absolute;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure: does this need a position: relative in the parent element?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think so...

@jonasfj jonasfj merged commit cfedb51 into dart-lang:master Oct 3, 2024
32 checks passed
@jonasfj jonasfj deleted the improved-banner-css branch October 3, 2024 10:07
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.

2 participants