Closed
Bug 771494
Opened 13 years ago
Closed 12 years ago
[HiDPI] Consider using highres images for about:config, network error, phishing and malware, etc pages
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: phiw2, Assigned: phiw2)
References
Details
(Whiteboard: [camino-2.1.3])
Attachments
(4 files, 4 obsolete files)
1.70 KB,
text/plain
|
alqahira
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details |
92.27 KB,
image/png
|
Details | |
58.45 KB,
application/zip
|
alqahira
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details |
62.89 KB,
image/png
|
Details |
The icon looks poor under HiDPI. We don't have much possibilities to load adaquate images depending on resolution. The most simple way is to use a large image, and scale it using the -moz-background-size property.
(technically we could use a 'resolution' / 'min-resolution' media query; however it is not implemented per spec in Gecko 1.9.2).
Attachment #639640 -
Flags: review?(alqahira)
![]() |
Assignee | |
Comment 1•13 years ago
|
||
Add control over the image size with the -moz-background-size peoperty to config.css and netError.css
Attachment #639641 -
Flags: review?(alqahira)
![]() |
Assignee | |
Comment 2•13 years ago
|
||
Assignee: nobody → phiw
![]() |
Assignee | |
Comment 3•13 years ago
|
||
A couple of minor nits:
1) The border around the inside and outside of the "compass ring" is a lot lighter; we seem to have various icons/images with different thicknesses/darknesses, so it always seems a crapshoot. This one might match the current state of the Camino icon better, so I'm not really hung up about it, more of an observation. It still looks OK at 1x, I think.
2) The outer edge/border of the warning triangle is thicker/darker; it's most noticeable on the triangle's bottom side.
3) The old triangle had a slight orange tint to the inner edge, which appears to be gone. (It looks like more of the orange, though maybe not all of it, is still there in the error page site icon from bug 770808?)
4) The . in the exclamation point appears too close to the bottom edge; the spacing was much better (and left the dot more distinct) in the old version.
Item 4 is really my biggest concern; combined with the slight blurriness that scaling down triggers in 1x mode, it's not attractive. Item 2 bothers me the second-most (though in this case it appears more of a 2x issue; dunno how it looks on a real 2x screen).
![]() |
Assignee | |
Comment 5•13 years ago
|
||
1. Last time I touched those 2 icons was in bug 505334 and bug 483360. At the time I used a bitmap form the appicon (the 128px one, slightly resized). This time, I used the vector source and scaled as needed. That would certainly explain the differences.
2. Ok, will tweak.
3. Right, I worked with the vector file I made for bug 770808, that icon has a straight gradient from more orange to more yellowish. I'll try to reconstruct the multiple gradients in the triangle.
4. I'll tweak.
And yeah, at low res there is probably some slight blurriness, can't really avoid it, unfortunately. I might try to sharpen slightly, but it could make things worse…, particularly on not-so-new MBP displays.
![]() |
Assignee | |
Comment 6•13 years ago
|
||
With updated warning.png
Attachment #639640 -
Attachment is obsolete: true
Attachment #639642 -
Attachment is obsolete: true
Attachment #639640 -
Flags: review?(alqahira)
Attachment #640140 -
Flags: review?(alqahira)
![]() |
Assignee | |
Comment 7•13 years ago
|
||
Comment on attachment 639641 [details]
patch to css files
Sorry, meant to r=ardissone the CSS part last week; it clearly behaves as expected :-)
Attachment #639641 -
Flags: review?(alqahira) → review+
Comment on attachment 640140 [details]
images, v1.1
Is it possible to make the dot of the ! a bit bigger without re-triggering problem 4 from comment 4? It seems just a little too small. Other than that, to the naked eye, the new icon appears nearly identical to the old one at 1x :-)
If it's not possible to make the dot bigger without it running into the other elements again, then let's take these as-is.
![]() |
Assignee | |
Comment 10•13 years ago
|
||
> Is it possible to make the dot of the ! a bit bigger without re-triggering
> problem 4 from comment 4? It seems just a little too small. Other than
> that, to the naked eye, the new icon appears nearly identical to the old one
> at 1x :-)
Perhaps ?
The original icon had a sort-of-rectangular dot. This looked pretty poor (nearly comically so) once resized. I made it a real dot (circle). I can see how it eventually looks to small at low-dpi.
Does v1.2 looks better ?
Attachment #640140 -
Attachment is obsolete: true
Attachment #640140 -
Flags: review?(alqahira)
Attachment #640508 -
Flags: review?(alqahira)
![]() |
Assignee | |
Comment 11•13 years ago
|
||
Attachment #640141 -
Attachment is obsolete: true
Comment on attachment 640508 [details]
Images, v1.2
> Does v1.2 looks better ?
Yes. r/icon-sr=ardissone (and sorry for the lag, again).
Attachment #640508 -
Flags: review?(alqahira) → review+
![]() |
Assignee | |
Updated•13 years ago
|
Attachment #639641 -
Flags: superreview?(stuart.morgan+bugzilla)
![]() |
Assignee | |
Updated•13 years ago
|
Attachment #640508 -
Flags: superreview?(stuart.morgan+bugzilla)
Comment 13•12 years ago
|
||
Comment on attachment 639641 [details]
patch to css files
sr=smorgan
Attachment #639641 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Updated•12 years ago
|
Attachment #640508 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
http://hg.mozilla.org/camino/rev/e8d549de8b6d
I intended to land the whole bunch under auspices of icon-sr when I got a moment to watch the tree, which didn't happen right away, but then I'd forgotten the bug by the time I next had occasions when I could tree-watch :-(
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [camino-2.1.3]
You need to log in
before you can comment on or make changes to this bug.
Description
•