bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Retina Display (HiDPI) not supported on SeaMonkey on MacOS X

RESOLVED FIXED in seamonkey2.52

Status

SeaMonkey
General
RESOLVED FIXED
5 years ago
a year ago

People

(Reporter: Cyrus Harmon, Assigned: Matt Felix)

Tracking

Trunk
seamonkey2.52
x86
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
Created attachment 720830 [details] [diff] [review]
seamonkey-hidpi.patch

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_2) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.152 Safari/537.22

Steps to reproduce:

Retina Display (HiDPI) is supported on Firefox (?), but not SeaMonkey. Bug 785330 tracks overall support and there may be other issues, but enabling HiDPI mode seems to make things work OK. The attached path enables HiDPI mode.


Actual results:

Hi resolution mode should isn't enabled.


Expected results:

Hi resolution mode should should be enabled.

Comment 1

5 years ago
Comment on attachment 720830 [details] [diff] [review]
seamonkey-hidpi.patch

Hi! Thanks for your patch. Without 2x graphics for our themes our toolbar buttons might appear rather blurry won't they? Perhaps we need a new icon set as well for OSX.
Attachment #720830 - Attachment is patch: true
Attachment #720830 - Flags: review?(stefanh)
Attachment #720830 - Flags: feedback?(neil)
Attachment #720830 - Flags: feedback?(iann_bugzilla)

Comment 2

5 years ago
Comment on attachment 720830 [details] [diff] [review]
seamonkey-hidpi.patch

Sorry, I have no idea what's going on here. I don't even know whether those are supposed to be spaces or tabs.
Attachment #720830 - Flags: feedback?(neil)
Confirming for now, looks like a bug.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 4

5 years ago
Cyrus,

Thanks for the patch. I'll need to see if I can simulate HiDPI mode somehow since I don't have a retina mac. Perhaps you could attach a screenshot showing how our icons looks with/without HiDPI mode?

Interestingly, Firefox doesn't make use of NSHighResolutionCapable. Instead, they have a NSPrincipalClass entry.

Updated

5 years ago
Summary: Retina Display (HiDPI) not supported on SeaMokney on MacOS X → Retina Display (HiDPI) not supported on SeaMonkey on MacOS X

Comment 5

5 years ago
We probably also need something like Bug 849460 
https://hg.mozilla.org/mozilla-central/rev/3dac95162a78
See Also: → bug 849460

Updated

5 years ago
See Also: bug 849460

Updated

5 years ago
See Also: → bug 849460

Comment 6

5 years ago
I think we should use the NSPrincipalClass entry. We also need to fix a bunch of icons.

Comment 7

5 years ago
> We also need to fix a bunch of icons.
I'm putting this down before I forget about this:

google.xml search icon:
http://hg.mozilla.org/mozilla-central/rev/f4beba4909b0#l6.12

image-rendering: -moz-crisp-edges;
http://hg.mozilla.org/mozilla-central/rev/f4beba4909b0#l7.11

Comment 8

5 years ago
Comment on attachment 720830 [details] [diff] [review]
seamonkey-hidpi.patch

I agree with stefanh, we probably need to use NSPrincipalClass
As this needs changes and some icons/css f-
Attachment #720830 - Flags: feedback?(iann_bugzilla) → feedback-

Comment 9

5 years ago
Comment on attachment 720830 [details] [diff] [review]
seamonkey-hidpi.patch

Per comment #6
Attachment #720830 - Flags: review?(stefanh) → review-

Comment 10

4 years ago
Should start by porting the two patches from Bug 764083
Just a quick update on this. Until a couple of versions ago (2.25?), adding

<key>NSHighResolutionCapable</key>
<true/>

at the end of info.plist worked pretty well, if not for the icons at least for the text. Starting with the release previous to 2.29 this doesn't work anymore. Any idea why and which workaround we can use now at least for the text?

Comment 12

4 years ago
You don't see any difference when using the NSPrincipalClass entry?
I'm not 100% sure about NSPrincipalClass, but having a look at Firefox's info.plist I've added this to the Seamonkey one:

<key>NSPrincipalClass</key>
<string>GeckoNSApplication</string>

but it changes nothing. Should have I done something different?

Comment 14

4 years ago
You should see a difference in text rendering (in the title bar, for example).
That's the point - I don't.
Don't ask me why, but updating from 10.9.2 to 10.9.4 (or maybe it's the rebooting that did the trick?) solved the issue. Now the text is ok.

Comment 17

4 years ago
I think the first step here is to add the key NSPrincipalClass, so if you can provide a patch for it, I'll be happy to review it :-)

Comment 18

3 years ago
Created attachment 8657334 [details] [diff] [review]
Patch to allow correct rendering on retina display

Here's the patch that fixes font display on retina screens. There are some blurry icons still, but at least browser is usable.

Comment 19

3 years ago
Comment on attachment 8657334 [details] [diff] [review]
Patch to allow correct rendering on retina display

Thanks! This looks good, but in order to apply the patch we need a better format. Do you have mercurial installed? If you have and have a cloned comm-central repo, you can set up (in your ~/.hgrc file) the right format like this:

[diff]
git = 1
showfunc = 1
unified = 8
----------------
And then just do hg diff > mypatch.diff (or mypatch.patch) after you've changed the file. There are lots of mozilla guides on how to set up mercurial, but you should be fine for now with just the above settings.

In this case, a git style patch in unified format from the top source dir would also be fine.

You can ask me for review (just set the review flag to ? and put my email address in the requestee field) when you upload the new patch. I will r+ it and then I'll check in the patch for you.
(Assignee)

Comment 20

a year ago
Created attachment 8853698 [details] [diff] [review]
HiDPI support patch

Hi! I tried to format the patch the way you suggested.
(Assignee)

Comment 21

a year ago
Comment on attachment 8853698 [details] [diff] [review]
HiDPI support patch

Added review request
Attachment #8853698 - Flags: review?(stefanh)

Updated

a year ago
Assignee: nobody → matthieufelix

Comment 22

a year ago
Comment on attachment 8853698 [details] [diff] [review]
HiDPI support patch

Thanks, this looks good!

-</dict>
+        <key>NSPrincipalClass</key>
+        <string>GeckoNSApplication</string>
+    </dict>
 </plist>

The indentation is a bit off here, but I can fix that on check-in.
Attachment #8853698 - Flags: review?(stefanh) → review+

Comment 23

a year ago
https://hg.mozilla.org/comm-central/rev/1b7e88473e8331887940b597e0b8d18607c05a0d
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED

Updated

a year ago
Target Milestone: --- → seamonkey2.50

Updated

a year ago
Target Milestone: seamonkey2.50 → seamonkey2.52
Stefan the bug is missing in the comment. Could you back this out and push it with an updated comment.
Flags: needinfo?(stefanh)

Updated

a year ago
Flags: needinfo?(stefanh)

Comment 26

a year ago
Created attachment 8858119 [details] [diff] [review]
Patch with correct commit message

This should be low-risk and is supposed to increase support for Retina screens, so I think it's worth taking it on branches.
Attachment #8858119 - Flags: approval-mozilla-esr52?
Attachment #8858119 - Flags: approval-comm-release?
Attachment #8858119 - Flags: approval-comm-beta?
Attachment #8858119 - Flags: approval-comm-aurora?

Updated

a year ago
Attachment #8858119 - Flags: approval-mozilla-esr52?
You need to log in before you can comment on or make changes to this bug.