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 URLs if external in media replace menu #18992

Closed
draganescu opened this issue Dec 7, 2019 · 8 comments
Closed

Always show URLs if external in media replace menu #18992

draganescu opened this issue Dec 7, 2019 · 8 comments
Assignees
Labels
[Feature] Media Anything that impacts the experience of managing media [Type] Enhancement A suggestion for improvement.

Comments

@draganescu
Copy link
Contributor

Is your feature request related to a problem? Please describe.
#16200 introduces a media replace menu which has the possibility to use URLs for media. These URLs can be external (e.g. an Unsplash picture) or internal (some image from the Media Library). However, there is a difference between the two:

  • when the image is from the media library, the URL is less important. Also it is of a somewhat higher probability to want to change that image with another from the Media Library
  • when the image is from an external URL then you should be made aware about this, instead of hunting the menu for that information, and it is also more likely to want to replace that image with another external image.

Describe the solution you'd like
When the current media's URL is external the URL viewer should be always open, un-collapsible via the Insert From URL menu item. When the current media's URL is internal the URL viewer should be collapsible and should hide when the menu is toggled.

Props @talldan for the idea.

@draganescu draganescu added [Feature] Media Anything that impacts the experience of managing media [Type] Enhancement A suggestion for improvement. Needs Design Feedback Needs general design feedback. labels Dec 7, 2019
@karmatosed
Copy link
Member

I'd love to dig a bit more into why the URL is needed. From an end-user perspective it's 'an image' and where comes from likely won't matter as much. What is the usability reason for this idea, what problem does it solve?

@draganescu
Copy link
Contributor Author

The URL is there because you have the option to insert media via URL, therefore, local media and remote media can be swapped only by URL.

@shaunandrews
Copy link
Contributor

If I'm reading this right, you're suggesting that the URL should always be shown when the Replace button is clicked:

image

If that's true, then I agree.

--

Its usually helpful to include visuals to help get everyone on the same page right from the get-go without the need for a back-and-forth comment thread.

@karmatosed
Copy link
Member

Ah ha! Ok that makes sense I was confused it was going to show by the actual image. @draganescu is @shaunandrews correct?

@shaunandrews
Copy link
Contributor

A few options for how this could work:

image

@draganescu
Copy link
Contributor Author

draganescu commented Dec 10, 2019

yes @Tammie that's what this is about. sorry for the confusion, I created this issue too quickly.

@shaunandrews I think the second option ...

second-option-insert-url

is best but, is there a reason for having the insert from URL option anymore?

no-insert-url

@shaunandrews
Copy link
Contributor

is there a reason for having the insert from URL option anymore?

Really just consistency, I imagined that option would do the same thing as Change. I'm not sure I have a strong opinion on this based on the design — when/if we have something to actually use I'm sure I'll develop an opinion.

@draganescu draganescu self-assigned this Dec 12, 2019
@karmatosed karmatosed removed the Needs Design Feedback Needs general design feedback. label Dec 18, 2019
@karmatosed
Copy link
Member

Removing design feedback as this now has a direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Media Anything that impacts the experience of managing media [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants