Opened 13 years ago
Closed 13 years ago
#20737 closed enhancement (fixed)
Improve appearance of "choose from library" link for headers and backgrounds
Reported by: | sabreuse | Owned by: | |
---|---|---|---|
Milestone: | 3.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | UI | Keywords: | has-patch |
Focuses: | Cc: |
Description
As seen <a href="http://wordpress.com/wp-admin/images/screenshots/flex-header-media-library.png">here</a>, the link we added to choose headers and backgrounds from the media library looks bad. This puts "or choose from image library" inline so it all flows better.
Attachments (7)
Change History (20)
#1
@
13 years ago
- Component changed from General to UI
- Keywords has-patch added
- Milestone changed from Awaiting Review to 3.4
#4
@
13 years ago
Perhaps we should also escape $image_library_url
(done in 20737.3.diff).
In media_buttons()
, get_upload_iframe_src()
is escaped:
http://core.trac.wordpress.org/browser/tags/3.3.2/wp-admin/includes/media.php#L376
#5
@
13 years ago
I can't see what the new patch makes it look like (not on a computer with svn etc right now), but one concern I'd have is that if it's inline the width would be an issue on smaller screens/devices. It's pretty late in the game to change layout stuff with no testing since we're hopefully doing rc1 today.
#6
@
13 years ago
Here's what it looks like with the patch: 20737.after.png.
#7
@
13 years ago
"or" on its own line is how we have it in the uploader. Inline is how we do it for internal links. Thinking leave for now, standardize all the "or"s in 3.5 instead.
#8
@
13 years ago
- Milestone changed from 3.4 to Future Release
- Type changed from defect (bug) to enhancement
#9
@
13 years ago
- Milestone changed from Future Release to 3.4
It is on its own line in the uploader only because it is a larger font and a better visual. Here, it is simply super ugly. This was reported by sabreuse at the request of ryan and me.
More importantly, though, this string is improperly internationalized. (The uploader isn't much better.) We should make a change here regardless.
#10
@
13 years ago
20737.4.2.diff is an attempt to use a consistent formatting in both custom-background.php
and custom-header.php
while keeping the same appearance as on a-tale-of-two-sources.png.
There's an odd space in the
<a>
tag in the first part of the patch:< a id="...">
.