Opened 12 years ago
Closed 12 years ago
#22609 closed defect (bug) (fixed)
IE7 + Media
Reported by: | nacin | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | audit has-patch commit |
Focuses: | Cc: |
Description
See #22446. There are a few remaining issues for IE7. Are these insurmountable? Are these worth it? Should we just serve the same experience we'd serve as if you had JS disabled (no "Add Media"?).
Attachments (2)
Change History (9)
#2
follow-up:
↓ 4
@
12 years ago
ie.css isn't going to be loaded on the frontend.
To confirm, these are no longer an issue?
- IE 7 needs some attention for the gallery view, see ticket:22446:22446.ie7_gallery.png
- IE 7 doesn't display the selected images (ticket:22446:22446.ie7_selected_images.png)
- In IE 7 Embed from URL doesn't work. Nothing happens after inserting an URL.
#3
@
12 years ago
22609.2.diff has 3 CSS hacks which will just target IE7, and work on both the front-end and in the admin. Can't think of any other way to do this at the moment. Suggestions?
#4
in reply to:
↑ 2
@
12 years ago
Replying to nacin:
ie.css isn't going to be loaded on the frontend.
To confirm, these are no longer an issue?
- IE 7 needs some attention for the gallery view, see ticket:22446:22446.ie7_gallery.png
This is fixed for me with 22609.2.diff
- IE 7 doesn't display the selected images (ticket:22446:22446.ie7_selected_images.png)
We can fix this, but it will add a load of CSS hacks to the media-view.css file. Since it doesn't really break major functionality for users, I'd vote to leave it out for IE7.
- In IE 7 Embed from URL doesn't work. Nothing happens after inserting an URL.
Works for me.
I went through and retested everything in IE7.
My IE7 checklist:
As far as I can see, everything now works in IE7.
2 minor changes in 22609.diff: