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

Replace image URLs with media library picker everywhere #11070

Open
hedgefield opened this issue Sep 21, 2018 · 7 comments
Open

Replace image URLs with media library picker everywhere #11070

hedgefield opened this issue Sep 21, 2018 · 7 comments

Comments

@hedgefield
Copy link
Contributor

hedgefield commented Sep 21, 2018

There are several locations in the plugin where we ask you to supply a URL in order to add an image. We want to replace those with the standard choose from media library button.

Locations this affects:

  • Search Appearance > General > Company logo
  • Config wizard step 4
  • Social > FB, Twitter, Pinterest default

Please let me know if I forgot any.

Design is coming.

For reference, how WP/Gutenberg does it:

screen shot 2018-09-21 at 11 20 49

How the WP media library does it:

screen shot 2019-01-07 at 11 27 36

screen shot 2019-01-07 at 11 28 05

@hedgefield
Copy link
Contributor Author

hedgefield commented Sep 24, 2018

One that I forgot to mention are the social previews in the metabox, but those already have a button that opens the Media Library. And for the social settings pages we have a pending redesign (#10708), which will take care of things there too.

So what's left is the Search Appearance setting and the config wizard:

Placing just a button would not have offered a way to preview what the image looks like, so I used the same pattern as our social preview editors. Clicking inside the square would open the media library, and so would the button. The dotted line makes it clearer that it's a placeholder zone. Can we do drag&drop here?

The size of the zone in this mockup is 400x240. I don't think we should go any bigger.

When an image is selected, the upload button is replaced by a replace + delete button. This mirrors how the Gutenberg featured image component works.

img upload zone 2

The button styles differ between the settings pages and the wizard, I've left that alone for now, but we have taken that into account in the config wizard redesign.

Also note that the Knowledge Graph inputs are now aligned below the headings, instead of next to them, which makes them consistent with the layout of the other settings in the paper.

cc @jdevalk @omarreiss @moorscode

@hedgefield
Copy link
Contributor Author

@afercia informed me that the decision within WordPress is to use the style of Gutenberg, without any additional buttons aside from the drop zone, so I've updated the mockup to match.

desktop copy 6

@afercia
Copy link
Contributor

afercia commented Jan 7, 2019

To clarify, the "drop zone" should be a styled button 🙂 Example from Gutenberg:

screenshot 2019-01-07 at 11 19 15

@hedgefield
Copy link
Contributor Author

hedgefield commented Mar 13, 2019

If I may propose one small tweak, to change the border and text color for this thing to #073CBA. I find the blue really makes it more clear it is an interactive element, especially when shown in close proximity to other buttons-styled-as-links in the Gutenberg context. It passes AAA WCAG with a 7.9:1 in comparison with the #F1F1F1 background color.

Screen Shot 2019-03-13 at 13 41 26

@afercia
Copy link
Contributor

afercia commented Mar 13, 2019

Just to confirm the pattern with the "grey area" being a button (with no additional "upload" button) is being used more and more in core, see https://core.trac.wordpress.org/ticket/43151 (will be in 5.2)

@hedgefield
Copy link
Contributor Author

hedgefield commented Oct 2, 2019

Made two definitive mockups to show both states, and how the alert could be included.

logo upload

logo preview

@luckickken
Copy link

@hedgefield let's not call this definitive yet. Not, until a product owner approves this on our design repo.

@Djennez Djennez removed the backlog label Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants