Opened 18 years ago
Closed 18 years ago
#4068 closed defect (bug) (fixed)
View Post Link Issues (RE: Changeset 5159)
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.2 | Priority: | low |
Severity: | minor | Version: | 2.2 |
Component: | Administration | Keywords: | |
Focuses: | Cc: |
Description
RE: [5159]
Oh where to begin...
- The right margin needs to be a fixed width rather than an
em
. On large resolutions, it moves onto of the sidebar:
- The CSS declaration should be in the stylesheet rather than inline so that admin skin replacements can change it's location
target="_blank"
? Tsk tsk. This isn't valid XHTML. ;)
Attachments (1)
Change History (9)
#2
in reply to:
↑ description
@
18 years ago
- Keywords needs-patch removed
- Owner changed from anonymous to rob1n
Replying to Viper007Bond:
RE: [5159]
Oh where to begin...
- The right margin needs to be a fixed width rather than an
em
. On large resolutions, it moves onto of the sidebar:
I can reproduce this. Fix coming up in a bit.
- The CSS declaration should be in the stylesheet rather than inline so that admin skin replacements can change it's location
Fixed in [5182].
target="_blank"
? Tsk tsk. This isn't valid XHTML. ;)
I'll talk to Matt.
#3
@
18 years ago
We'll need to adjust rtl.css too.
Something like
a.view-link { right: auto; left: 2em; margin-right: 0; margin-left: 19em; }
For what's currently in trunk. I didn't look at the attached patch :)
#4
@
18 years ago
Ah, I'll do that now.
And the patch doesn't completely work under IE (both 6 and 7). Still "functional," but not perfect ;).
#6
@
18 years ago
As for the target="_blank" thing, I'll talk to Matt about possibly using a JavaScripty window.open solution. But I'm not overtly concerned about internal HTML validation.
Note: See
TracTickets for help on using
tickets.
(In [5182]) Move inline styles to wp-admin.css. see #4068