#22752 closed defect (bug) (fixed)
Get media modal to play nice with P2 theme
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | low |
Severity: | normal | Version: | |
Component: | Media | Keywords: | has-patch |
Focuses: | Cc: |
Description
There are a few places where we can improve styles for the front-end media modal in the P2 theme.
Attachments (5)
Change History (19)
#2
@
12 years ago
22752.diff cleans up a few things:
- Removes a:hover border bottom
- Adjust layout of .link-to-custom field
- Adds some margin to the bottom of .post-upload-ui
- Fixes alignment for selected images (.selection-view)
The only thing that stood out to me that should get fixed in the theme is the add media button:
#3
@
12 years ago
in 22752.2.diff:
- I removed border-bottom: none; for a:hover (Helenyhou brought down her fist ;-) - this will need to be done in the P2 theme)
- Removed the .link-to-custom style as it displays differently cross-browser
- Changed the magic number (23px) to something a little less magic (2em)
#4
@
12 years ago
Just so I don't sound crazy, I brought down my fist because there is a line where we can't account for things, and it's best not to cross it. Where would it stop? All borders? Strikethrough? Background? Etc.
In any case, 22752.2.diff looks good to me. Doesn't seem to have adverse effects within the admin, either.
#6
@
12 years ago
Let's bring back border-bottom: none, per IRC. Otherwise, .2.diff.
Also, the select elements are squished. Maybe we should also normalize those the way we do in the admin? (Do we?)
#8
@
12 years ago
22752.3.diff:
- Adds border-bottom: none back in
- Attempts to fix select box formatting
#9
@
12 years ago
- Summary changed from Get media modal to place nice with P2 theme to Get media modal to play nice with P2 theme
#10
@
12 years ago
22741.4.patch looks good to me.
#11
@
12 years ago
Caught that select padding is actually 2px for the admin at large. .5 contains that little change.
Note that some of this may simply be P2's problem. But, if there's anything we're doing that is fairly silly or lame, we should fix it. A few reset rules didn't hurt anyone.