Opened 6 months ago

Closed 6 months ago

#22662 closed defect (bug) (fixed)

No way to access Media Link while adding/editing Media

Reported by: vickybiswas Owned by: koopersmith
Priority: normal Milestone: 3.5
Component: Media Version: 3.5
Severity: normal Keywords: has-patch
Cc: batmoo@…, gabriel.koen@…, merty92@…

Description

There is no way to access the link(url) for a media file in the new Media UI.

Submitting a patch to do the same.

Attachments (5)

Capture.PNG (148.6 KB) - added by vickybiswas 6 months ago.
A shot of how the patch places the url
add media url to Add Media.patch (827 bytes) - added by vickybiswas 6 months ago.
Patch to add a url
22662.patch (475 bytes) - added by merty 6 months ago.
22662.png (51.4 KB) - added by merty 6 months ago.
22662.diff (1.5 KB) - added by nacin 6 months ago.

Download all attachments as: .zip

Change History (13)

  • Cc batmoo@… added

comment:2 follow-up: ↓ 4   nacin6 months ago

  • Version set to trunk

Can you please re-patch without touching the minified CSS? That isn't necessary, and will end up rendering the patch stale really quick (if anything else changes in the file). Thanks!

I've been thinking about this for some time. Is the URL necessary? I'm not sure where it should go. It's useful to have it there if you need it, but it really gets in the way if you don't. Maybe show it near the metadata (dimensions, name, file type, etc.), style it like text but make it a readonly input?

  • Cc gabriel.koen@… added

comment:4 in reply to: ↑ 2   vickybiswas6 months ago

The URL is indeed needed for various purposes. I have tried to put it in the place where it will least come in the way. Seems your idea matches what I have done. Uploading a screenshot along with the repatch.

Replying to nacin:

Can you please re-patch without touching the minified CSS? That isn't necessary, and will end up rendering the patch stale really quick (if anything else changes in the file). Thanks!

I've been thinking about this for some time. Is the URL necessary? I'm not sure where it should go. It's useful to have it there if you need it, but it really gets in the way if you don't. Maybe show it near the metadata (dimensions, name, file type, etc.), style it like text but make it a readonly input?

A shot of how the patch places the url

Patch to add a url

  • Component changed from General to Media
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.5

This is great. Let's see if we can compress it a bit more with some CSS for this one field. (Patches welcome.) Assigning to koopersmith for eventual review.

  • Owner set to koopersmith
  • Status changed from new to assigned

merty6 months ago

merty6 months ago

  • Cc merty92@… added

nacin6 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 22965:

In the media modal, show the Link To: url.

Props nacin
fixes #22662

Note: See TracTickets for help on using tickets.