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

Meta boxes are hidden behind a scrollbar #19709

Closed
youknowriad opened this issue Jan 16, 2020 · 10 comments · Fixed by #20262
Closed

Meta boxes are hidden behind a scrollbar #19709

youknowriad opened this issue Jan 16, 2020 · 10 comments · Fixed by #20262
Assignees
Labels
[Feature] Meta Boxes A draggable box shown on the post editing screen [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@youknowriad
Copy link
Contributor

Note that this does seem to work properly in WordPress 5.3

To reproduce

  • Install and active Yoast
  • Create a new post
  • Notice that the Yoast metabox is hidden underneath, you need to scroll the page to reach it.

Capture d’écran 2020-01-16 à 6 45 53 PM

@youknowriad youknowriad added [Feature] Meta Boxes A draggable box shown on the post editing screen [Type] Bug An existing feature does not function as intended labels Jan 16, 2020
@youknowriad
Copy link
Contributor Author

youknowriad commented Jan 16, 2020

I feel like we go back and forth with this kind of issues.
If we try to fix it, we might break the Redirect behavior etc... cc @aduth @jasmussen

@aduth
Copy link
Member

aduth commented Jan 16, 2020

I recall doing a round of manual testing of meta boxes in the round of updates associated with #18732. From the screenshot, it seems to also have regressed #18873 (note the white gap above the meta border and below the beige canvas).

But regarding the fact there is an initial scroll... it might be expected? At least, as part of the changes introduced with #16460, a 50vh minimum height is now applied to the editable area, which could be expected to cause some scroll.

@jasmussen
Copy link
Contributor

A jargony term is that the toolbar is "below the fold". I personally think this is correct behavior.

@youknowriad
Copy link
Contributor Author

If you don't have "scrollbar always visible", you might not even notice that you have metaboxes, for me this is a bug that should be fixed.

@jasmussen
Copy link
Contributor

I have opinions on that option existing in the first place, but accept that it is the state of the world, and I do see your point.

@aduth
Copy link
Member

aduth commented Jan 17, 2020

It's not clear to me what the fix would be. If the expectation with the typewriter implementation is that we always have a vertically-tall "gap" (which I think was used to intentionally force the scroll), then how can we expect a fix to not conflict with this?

Some ideas:

  • Shorter minimum height for the click redirect
  • Adjusted minimum height for the click redirect on the basis of whether meta boxes exist
  • Unset minimum height for the click redirect when there are no paragraphs yet in the editor
    • Might lead to unexpected "jumpiness".
  • Display meta boxes as "fixed" to the bottom of the screen
    • I seem to recall this was how it behaved in earlier implementations, and that we had some issues where meta boxes could occupy too much of the page and overlap with the block list. There could be some compromise here in implementing this and assigning a maximum height to the meta box area.

cc @ellatrix

@ellatrix
Copy link
Member

I'm also not sure what a fix would look like. The space at the bottom is needed for the typewriter effect, but it doesn't need to be taken up by the click redirect element. The only requirement is that there is some space (50vh I believe). Perhaps we could try setting the height of the click redirect in JS rather than CSS.

Personally I don't really mind the meta boxes showing under the fold (especially since it's legacy UI). Perhaps we can have a little label sticking out at the bottom to make it clear that they're there.

@jasmussen
Copy link
Contributor

If the challenge is purely that the metaboxes are invisible when below the fold with autohiding scrollbars, a classic fix is to simply ensure that they are just above the fold, so they indicate you can scroll. So the fix could be as "simple" as using <100% of the viewport height for the editing canvas.

@jorgefilipecosta
Copy link
Member

If the challenge is purely that the metaboxes are invisible when below the fold with autohiding scrollbars, a classic fix is to simply ensure that they are just above the fold, so they indicate you can scroll. So the fix could be as "simple" as using <100% of the viewport height for the editing canvas.

Hi @jasmussen, would you be able to propose a PR trying this fix?

@jasmussen
Copy link
Contributor

Done!

@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Feb 17, 2020
jasmussen added a commit that referenced this issue Feb 21, 2020
* Try: Show metaboxes peeking in even on tiny screens.

Fixes #19709.

* Try fixing the e2e test

Co-authored-by: Riad Benguella <benguella@gmail.com>
jorgefilipecosta pushed a commit that referenced this issue Feb 24, 2020
* Try: Show metaboxes peeking in even on tiny screens.

Fixes #19709.

* Try fixing the e2e test

Co-authored-by: Riad Benguella <benguella@gmail.com>
jorgefilipecosta pushed a commit that referenced this issue Feb 24, 2020
* Try: Show metaboxes peeking in even on tiny screens.

Fixes #19709.

* Try fixing the e2e test

Co-authored-by: Riad Benguella <benguella@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Meta Boxes A draggable box shown on the post editing screen [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants