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

Full-height button in Cover: move to dropdown and make Fullscreen #27496

Closed
jasmussen opened this issue Dec 4, 2020 · 5 comments
Closed

Full-height button in Cover: move to dropdown and make Fullscreen #27496

jasmussen opened this issue Dec 4, 2020 · 5 comments
Assignees
Labels
[Block] Cover Affects the Cover Block - used to display content laid over a background image [Type] Enhancement A suggestion for improvement.

Comments

@jasmussen
Copy link
Contributor

The Cover block has a cool new "full height" button, which sets its height to 100vh in the sidebar controls, and acts as a convenenient toggle:

cover

Upon testing this, it seems like you'd nearly always want to combine the 100vh height with full-wide. And in doing so, it could be grouped along with the other alignments/positions. Like so:

Screenshot 2020-12-02 at 10 44 13

That would also allow us, separately from this effort, to expand the feature to more blocks than just Cover.

@jasmussen jasmussen added [Type] Enhancement A suggestion for improvement. [Block] Cover Affects the Cover Block - used to display content laid over a background image labels Dec 4, 2020
@ntsekouras
Copy link
Contributor

And in doing so, it could be grouped along with the other alignments/positions. Like so:

No strong opinion here but I think we shouldn't combine fullscreen with alignment options as they are different things, even if fullscreen and wide could be usually combined.

@retrofox retrofox self-assigned this Dec 4, 2020
@retrofox
Copy link
Contributor

retrofox commented Dec 4, 2020

And in doing so, it could be grouped along with the other alignments/positions. Like so:

No strong opinion here but I think we shouldn't combine fullscreen with alignment options as they are different things, even if fullscreen and wide could be usually combined.

Yeah... I have a similar feeling but don't have a strong opinion either. In some way, we're going to lose the ability to combine full height with all other alignments.

@jasmussen
Copy link
Contributor Author

Excellent question to bring up and work out here in the thread.

The point of the issue is that you'll almost never want to have just full-height (and if you do, you can in the sidebar with 100vh). You'll almost always want full height and full width. And you can't do 100vw and vh inside post content. So the only solution to getting actual fullscreen, is to combine full-wide with 100vh.

To me, that means the button could sit right below full-wide and toggle both.

@paaljoachim
Copy link
Contributor

paaljoachim commented Jan 30, 2021

Let's see here... thinking out loud.
We have a "Change Alignment" drop down.

Align left - pushes to the left.
Align center - pushes to center.
Align right - pushes to the right.
Wide width - creates an extra wide Cover block.
Full width - pushes to the way to the left and right.

Similar to the Full width alignment we have a Full height toggle. As we have the Full width inside the alignment drop down it seems fitting to also have a Full height alignment in the same drop down. One could then do a multi select and select to have Full width and a Full height. Like so. (I have made some adjustments to the current icons. To signal full width and full height.)

Cover-block-Full-width-Full-height

Btw
Another alignment I really would like to add is a default width that is by default always selected. So a user can see the default width is selected and then choose something else if they want to. #20941
I do think we discussed this in another issue a while ago. Arguments were brought up why my suggestion was not a great idea. That is though a sidetrack to moving the Full-height button to the alignment drop down issue we are discussing here.

@jasmussen
Copy link
Contributor Author

Revisiting this one in light of many of the alignment refactors that have happened since this conversation, I actually tend to think we should keep the button as-is, at least for now, and then revisit/reopen if a need arises.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Cover Affects the Cover Block - used to display content laid over a background image [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

4 participants