Text Zoom dialog should round value, not truncate it

VERIFIED FIXED in mozilla0.9.9

Status

SeaMonkey
UI Design
--
trivial
VERIFIED FIXED
17 years ago
10 years ago

People

(Reporter: Matthew Paul Thomas, Assigned: jag (Peter Annema))

Tracking

Trunk
mozilla0.9.9

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

994 bytes, patch
stephend@netscape.com (gone - use stephen.donner@gmail.com instead)
: review+
Simon Fraser
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

17 years ago
Build: 2001122508, Mac OS 9.1

To reproduce:
1.  Choose `View' > `Text Zoom' > `Other...'.
2.  Enter `99.9' (without quotes), and hit OK.
3.  Open the `View' menu to see what the current zoom level is.

What you should see:
*   100 %.

What you actually see:
*   99 %.
->jag
Assignee: blakeross → jaggernaut
(Assignee)

Updated

17 years ago
OS: Mac System 9.x → All
Hardware: Macintosh → All
Target Milestone: --- → mozilla0.9.8
(Assignee)

Comment 2

17 years ago
Created attachment 65542 [details] [diff] [review]
parseFloat, not parseInt

Comment 3

17 years ago
Comment on attachment 65542 [details] [diff] [review]
parseFloat, not parseInt

sr=sfraser
Attachment #65542 - Flags: superreview+
Comment on attachment 65542 [details] [diff] [review]
parseFloat, not parseInt

r=stephend
Attachment #65542 - Flags: review+
(Assignee)

Comment 5

17 years ago
-> 0.9.9
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Comment 6

17 years ago
Checked in.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
verified fixed using 2002.01.29.1x comm bits on linux rh7.2, mac 10.1.2 and
win2k. tested with 99.9, 97.5 and 82.3, which properly rounded to 100, 98 and
82, respectively.
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite

Updated

10 years ago
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.