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

Always show "new gallery item" below the gallery #7300

Closed
mtias opened this issue Jun 13, 2018 · 7 comments · Fixed by #7391
Closed

Always show "new gallery item" below the gallery #7300

mtias opened this issue Jun 13, 2018 · 7 comments · Fixed by #7391
Labels
[Feature] Blocks Overall functionality of blocks [Feature] Media Anything that impacts the experience of managing media [Type] Task Issues or PRs that have been broken down into an individual action to take
Milestone

Comments

@mtias
Copy link
Member

mtias commented Jun 13, 2018

Currently, selecting a gallery can shift the layout presentation to accommodate the extra placeholder item. That makes it hard to work with galleries since layout can change on selection and not be faithful.

Let's move the "new gallery item" to be always below the gallery, full-width, and avoid disrupting the gallery presentation.

image

@mtias mtias added [Type] Task Issues or PRs that have been broken down into an individual action to take [Feature] Blocks Overall functionality of blocks [Feature] Media Anything that impacts the experience of managing media labels Jun 13, 2018
@ZebulanStanphill
Copy link
Member

Just to be clear, the placeholder will still only appear when the block is selected, right? If so, then great! The placeholder moving the other images around felt a bit odd.

@mtias
Copy link
Member Author

mtias commented Jun 13, 2018

Yes, correct, only on selection. Just would avoid the gallery becoming 3 columns for some reason on selection.

@karmatosed
Copy link
Member

I sort of like it shifting I have to say. I did say in PR but saying here to make sure say in right place, sorry about that.

For example here where it shows there could be more.

2018-06-20 at 20 41

I feel it gives more of an impression of what it could be.

@tofumatt
Copy link
Member

It's harder to locate though; putting the inserter in a common place and on its own line makes it easier to locate and makes it stand out. I actually found, just in testing this patch against the behaviour in master, that the existing, inline inserter is harder to locate/tell apart (from those example, grey-background images), and the placement being inconsistent makes it annoying to add images one after the other.

I think the full width, consistently-placed inserter is much better and more discoverable.

@caxco93
Copy link
Contributor

caxco93 commented Jun 20, 2018

Besides from the inserter being more discoverable, I think another great benefit is preventing the layout from changing and not being faithful to what the user would finally see as posted.

Regards

I feel it gives more of an impression of what it could be.

I see it more as a "what WILL the user see posted" vs "what COULD the user see posted"

@mtias
Copy link
Member Author

mtias commented Jun 22, 2018

I feel it gives more of an impression of what it could be.

The problem is that selecting a gallery to, say, add a caption to an image, can drastically change the layout, which makes it very hard to write the caption and see how it would look because the "add new image" is changing the whole layout.

@karmatosed
Copy link
Member

After reflecting on this 👍 lets get it in and tested. I do still feel the impression point is valid but consensus is also ok to go with.

@tofumatt tofumatt added this to the 3.2 milestone Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Feature] Media Anything that impacts the experience of managing media [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants