Closed
Bug 844725
Opened 11 years ago
Closed 11 years ago
click-to-play: new placeholder content has some styling issues
Categories
(Core Graveyard :: Plug-ins, defect, P3)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: djc, Unassigned)
References
Details
Attachments
(1 file)
25.01 KB,
image/png
|
Details |
- The "close" button should have the same margin on the right and on top - The "Activate Adobe Flash." text should be further down from the play button - I don't think a '.' at the end of a sentence fragment makes sense
Updated•11 years ago
|
Priority: -- → P3
Blocks: click-to-play
Comment 1•11 years ago
|
||
Can you post a screenshot of http://people.mozilla.com/~shorlander/ctp-test/ctp-test.html ? I suspect that the close button is in the correct place, and just the plugin object is offset in a weird way. Shorlander will have to comment on the other things. See also bug 840414 which will probably fix the text positioning issue.
Depends on: 840414
Comment 2•11 years ago
|
||
Yeah it looks like entire plugin container is getting clipped. I agree with removing the period.
Reporter | ||
Comment 3•11 years ago
|
||
Yeah, I noticed the close button is placed correctly in the placeholder on other sites (including the test page from comment 1). FWIW, the page where I previously took the screenshot also no longer has the weirdly placed close button. So I guess we can close this in favor of bug 840414, if the text placement (and the period) also get addressed there? Or would you prefer keeping the separate bug?
Updated•11 years ago
|
Flags: needinfo?(shorlander)
Comment 4•11 years ago
|
||
(In reply to Dirkjan Ochtman (:djc) from comment #3) > Yeah, I noticed the close button is placed correctly in the placeholder on > other sites (including the test page from comment 1). FWIW, the page where I > previously took the screenshot also no longer has the weirdly placed close > button. So I guess we can close this in favor of bug 840414, if the text > placement (and the period) also get addressed there? Or would you prefer > keeping the separate bug? Closing this since it appears to be site specific. We are doing additional work on the strings and revising the visual design now that some of our goals have changed.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(shorlander)
Resolution: --- → WORKSFORME
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•