Closed
Bug 39117
Opened 25 years ago
Closed 23 years ago
[zoom]View Enlarge/Reduce Text Size doesn't work on mac
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: bugzilla, Assigned: pierre)
References
(Blocks 1 open bug, )
Details
(Keywords: access, helpwanted, platform-parity)
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
spun off from bug 19323. pierre, i'm taking a guess here as to whether this is
style system related. if you disagree, feel free to reassign (i don't know where
else this would go, cc'ing rginda and bill to see if they would.)
tested using opt commercial bits 2000.05.12.08 on Mac OS 9.0.
to repro:
1. load the above url.
2. select View > Enlarge Text Size.
result: web content shifts a bit to the left, but the text size doesn't change.
3. repeat step 2 (several times).
result: nothing changes at all.
4. exit and restart the browser.
5. load the above url.
6. select View > Reduce Text Size.
result: content moves a smidgen upwards, but no change to text size.
7. repeat step 6 (several times).
result: nothing changes at all.
Assignee | ||
Comment 1•25 years ago
|
||
CCd erik who did the implementation on Win and Unix. This should have been
extremely easy to implement, like on the other platforms, but it ended up in
something a bit puzzling. Basically, we need to do something like this:
float textZoom;
mDeviceContext->GetTextZoom(textZoom);
pixelSize = NSToIntRound(mFont->Size * textZoom * app2dev);
In nsFontMetricsMac::Init(), if we simply multiply 'theStyle.tsSize' with the
textZoom factor, it doesn't work. However if (in addition?) we multiply
mFont.size with textZoom, it works but then we leak thousands of FontMetrics.
Pushed to M17.
Status: NEW → ASSIGNED
Target Milestone: --- → M17
Comment 2•25 years ago
|
||
Please do not modify mFont itself, since the device context uses it in the font
cache. On Windows and Unix, I didn't touch mFont for that reason. And that is
also why we are doing the text zoom down in the platform-specific code. We could
have done the zoom in the XP code, but I think that would have necessitated a
copy of the nsFont before looking it up in the cache, and since the font cache
is accessed many times, I feared that this would affect performance. Currently,
the fonts are cached according to their un-zoomed nsFont.size, and the zooming
is done when we turn the logical nsFont into a physical font.
Major loss of functionality. Nominating nsbeta3.
Severity: normal → major
Keywords: nsbeta3
Comment 6•24 years ago
|
||
I filed bug 52969 to remove this menu item form the Mac builds, assuming that
this bug is not going to be fixed for nsbeta3/rtm (and since this bug hasn't
really moved in four months, I expect that this will not be fixed before ship).
Why doesn't this bug have a triage status?
Not fixing this would leave Mozilla without a very useful feature that IE has.
Comment 8•24 years ago
|
||
To be clear, I'm not advocating removing the menu items as preferable to
actually fixing this bug. However, if this bug is not fixed (and it may
not be fixed before nsbeta3 and rtm) then we should remove the menu items
for this feature.
Comment 9•24 years ago
|
||
Could we get a thumbs-up/thumbs-down for this bug being fixed in the
nsbeta3 and rtm time frames.
If it cannot be done, then we should move forward on the (regrettable)
alternative, which is to remove the associated Menu for 'Text Size'
from the Mac UI (bug 52969).
Keywords: rtm
Comment 11•24 years ago
|
||
Marking rtm- for Netscape folks. If the extenernal contributor can get this to
work, then we'll reconsider taking this, but we won't hold the release for it.
Whiteboard: [rtm+] → [rtm-]
Comment 12•24 years ago
|
||
So, how do we fix this, and who wants to take a stab?
Not having a Mac seriously reduces the chance of me fixing it ;-)
Assignee | ||
Comment 13•24 years ago
|
||
This menu is no longer on the branch (see bug 54965). Marking Future.
Target Milestone: M17 → Future
Nominating for Mozilla 0.9.
Keywords: mozilla0.9
Comment 15•24 years ago
|
||
Just to note, this menu is now explicitly disabled on the Mac since the
backend did not work. When this bug gets fixed, you will need to undo the
fix for bug 52969, and the menu should then Just Work (tm).
Reporter | ||
Updated•24 years ago
|
Keywords: relnoteRTM
Reporter | ||
Comment 16•24 years ago
|
||
the relnotes for netscape 6.0 (or, even better, its online help) should mention
that this feature is now gone (see bug 52969). cc'ing vera.
Comment 17•24 years ago
|
||
Adding kristif to cc list
Possibly include this info in "Migrating to N6 from Other Browsers" guide
Updated•24 years ago
|
Comment 18•24 years ago
|
||
Nom. nsbeta1. View Enlarge/Reduce Text Size is basic Nav1.x functionality we'd
like to have working on the Mac again, plus it's important for accessibility to
the visually impaired. Marking access.
Reporter | ||
Updated•24 years ago
|
Keywords: nsbeta3,
relnoteRTM,
rtm
Whiteboard: [rtm-] relnote-user → relnote-user
Comment 19•24 years ago
|
||
Netscape's standard compliance QA team reorganised itself once again, so taking
remaining non-tables style bugs. Sorry about the spam. I tried to get this done
directly at the database level, but apparently that is "not easy because of the
shadow db", "plus it screws up the audit trail", so no can do...
QA Contact: chrisd → ian
Comment 20•24 years ago
|
||
nominating for dogfood (from sdagley's list of bugs that are good candidates for
our next release)
Keywords: nsdogfood
Reporter | ||
Comment 21•24 years ago
|
||
this was futured back in oct 2000 --clearing milestone to renominate.
Target Milestone: Future → ---
Comment 22•24 years ago
|
||
Removing rel-note designation (which applied to N6) from Status Whiteboard, as
this bug is being considered for a fix in a 6.x release.
Whiteboard: relnote-user
This bug has been without a target milestone for over two months. Has this fallen off the appropriate radars?
Blocks: 31961
Reporter | ||
Comment 24•24 years ago
|
||
pierre isn't around...->attinasi?
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
Comment 25•24 years ago
|
||
The menu items are gone (still). We need to put them back and then get the
functionality in. I think it is best left for Pierre, assuming he returns.
Sadly, sending it back to Pierre since I will not have time for this over the
summer due to more pressing layout issues.
Assignee: attinasi → pierre
Reporter | ||
Updated•24 years ago
|
Keywords: helpwanted
*** Bug 80050 has been marked as a duplicate of this bug. ***
Updated•24 years ago
|
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Comment 28•24 years ago
|
||
pierre: ping? :-) Are there any plans to fix this soon? It's rather
embarrassing, and confusing for Mac people, who can't work out why everyone else
has it and they don't.
Gerv
Assignee | ||
Comment 29•24 years ago
|
||
I'll look into it again.
Status: NEW → ASSIGNED
Summary: View Enlarge/Reduce Text Size doesn't work on mac → [zoom]View Enlarge/Reduce Text Size doesn't work on mac
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9.4 → mozilla0.9.5
Assignee | ||
Comment 30•24 years ago
|
||
I guess I won't have any problem to get r/sr on that one... Simon?
Back to 0.9.4
Target Milestone: mozilla0.9.5 → mozilla0.9.4
Assignee | ||
Comment 31•24 years ago
|
||
Comment 32•24 years ago
|
||
r/sr=sfraser
Comment 33•24 years ago
|
||
r/sr means "either r or sr, take your pick". You still need another reviewer or
super-reviewer.
Comment 34•24 years ago
|
||
Wouldn't you be better off making this adjustment before the following (both for
rounding and because of the 9px cutoff)?:
short textSize = float(aFont->size) / dev2app;
if (textSize < 9 && !nsDeviceContextMac::DisplayVerySmallFonts())
textSize = 9;
Assignee | ||
Comment 35•24 years ago
|
||
David, the textzoom should be done at the end because:
- The cutoff is a hidden Mac-only feature that was used a long time ago when font
size issues were not resolved yet (bug 18136).
- In the current proposal for revisiting font issues (bug 74186), we agreed that
the text zoom should have no size limit and be calculated after the minimum font-
size calculation and the font-size-adjust, just before the actual value.
r/sr someone?
Comment 36•24 years ago
|
||
Comment 37•24 years ago
|
||
a=dbaron on behalf of drivers
Is this getting in? It would help me with bug 31961 if this change got in first.
Assignee | ||
Comment 39•23 years ago
|
||
Fixed checked in nsFontMetricsMac.cpp and viewZoomOverlay.js
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 40•23 years ago
|
||
Cool!
So what makes it possible to do this now?
Comment 41•23 years ago
|
||
Pierre, thank you sooo much for fixing this! It was one of the most worst Mac
shortcomings. This problem alone was keeping several people I know from using
Mozilla/NS6.
Comment 42•23 years ago
|
||
Just tried this out in today's Mac OS X trunk build (2001082905)
and the zooming/shrinking works great. There is a cosmetic
issue, though -- the menu item in the View menu doesn't change
to reflect the current size. The submenu correctly displays
the zoom, but the top level item always says "Text Size %100" regardless
of the current zoom level.
Should this be filed as a seperate bug?
Assignee | ||
Comment 43•23 years ago
|
||
Correct. The label stays at the value it has the first time you click on the
menu. For instance if you do twice Cmd-'-' after launching the app and click on
the menu, the label stays stuck at 75% (even though the zoom itself works fine
and the checkbox in the submenu shows the correct value).
This is Mac-only; Win98 doesn't show the problem. I guess the bug is in
updateViewMenu() in viewZoomOverlay.js. Please open a new bug and assign to me,
but if you or someone else wants to take a look, feel free to take it. Thanks!
Comment 44•23 years ago
|
||
pinkerton mentioned that the Mac OS X menu not changing is a known Mac OS X
problem. There's a bug on that somewhere (bug pink for it or query bugzilla).
Assignee | ||
Comment 45•23 years ago
|
||
I'm seeing the problem in OS 9.1
Comment 46•23 years ago
|
||
Hmmm. Well, I assert it's a problem in Mac specific code. The js front-end code
works fine on both Linux and Windows (with no special casing) and should just
work fine on Mac too. If you're seeing this in Mac OS 9.1, let's file a bug on
it and get it fixed :-)
Reporter | ||
Comment 47•23 years ago
|
||
i've filed bug 97549 for Text Size (100%) not updating.
vrfy fixed on both Mac OS X [2001.08.29.05-comm] and Mac OS 9.1
[2001.08.29.08-comm, emul over X]. tested both the shortcuts [cmd+plus/minus]
and selecting the menu choices.
yay!
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•