Skip to content

Make notices push down content#13614

Merged
jasmussen merged 3 commits intomasterfrom
try/restore-notices-push-content
Feb 14, 2019
Merged

Make notices push down content#13614
jasmussen merged 3 commits intomasterfrom
try/restore-notices-push-content

Conversation

@jasmussen
Copy link
Copy Markdown
Contributor

This PR restores the good stuff from #12301. That is: it allows notices to push down content. Dismissible notices are sticky at the top, non-dismisible notices scroll out of view.

This is mostly an exact copy of the other PR, but fresh. The behavior has a number of benefits:

  • If you have multiple non-dismissible notices, you can still actually use the editor.
  • Notices no longer cover the scrollbar.
  • Notices no longer cover the permalink interface.
  • Notices now only cover content if you do not dismiss the notices.

Fixes #7276.

GIFs:

desktop

mobile

screenshot 2019-01-31 at 11 58 01

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Notices /packages/notices [Type] Enhancement A suggestion for improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants