Opened 11 years ago
Closed 11 years ago
#28952 closed defect (bug) (invalid)
Media Grid: "Uploaded To" link is not clickable
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.0 |
Component: | Media | Keywords: | has-patch dev-feedback |
Focuses: | ui, javascript, administration | Cc: |
Description
In "Uploaded To" line in the media grid, post title is a link (see the screenshot).
When you click on it, the item gets selected, like when clicking on the rest of the text.
Attachments (2)
Change History (9)
#2
follow-up:
↓ 5
@
11 years ago
- Summary changed from Media Grid: "Uploaded To" link is not clickable to Media Grid: "Uploaded To" link and "Edit Media" icon are not clickable
Also the "Edit Media" icon (the pencil icon on the top left while hovering) is not clickable.
#4
follow-up:
↓ 6
@
11 years ago
This is because media.view.Attachment
binds all anchor clicks to preventDefault
here.
All anchors should be reviewed in this template (and any child classes that don't override the events hash). We should change any of these anchors to buttons that are not hyperlinks to avoid requiring preventDefault
triggered on click.
#5
in reply to:
↑ 2
@
11 years ago
- Summary changed from Media Grid: "Uploaded To" link and "Edit Media" icon are not clickable to Media Grid: "Uploaded To" link is not clickable
Replying to gauravmittal1995:
Also the "Edit Media" icon (the pencil icon on the top left while hovering) is not clickable.
Please don't double up on a ticket like that. There was a ticket open for that already: #28945.
#6
in reply to:
↑ 4
@
11 years ago
Replying to ericlewis:
All anchors should be reviewed in this template (and any child classes that don't override the events hash). We should change any of these anchors to buttons that are not hyperlinks to avoid requiring
preventDefault
triggered on click.
Great point by ericlewis, it's a bit weird to prevent all events than rewrite their actions.
Sorry, but don't know how to better implement that click in Backbone :P