Closed Bug 18774 Opened 25 years ago Closed 24 years ago

[feature] Increase/Decrease font size need to be written

Categories

(SeaMonkey :: Composer, defect, P3)

Product:
SeaMonkey
Component:
Composer
Platform:
x86
Windows NT
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: cmanske, Assigned: mozeditor)

References

Details

(Whiteboard: fix in hand) 

The HTML font size is always confusing to non HTML geeks, so use simple
"Increase Font Size" and "Decrease Font Size" buttons.
This should increase/decrease all text in a selection by 1 HTML unit,
adjusting existing "FONT SIZE=" values. Underlying code (nsHTMLEditor methods)
also need to be implemented.
Status: NEW → ASSIGNED
Target Milestone: M12
UI (the toolbar buttons) is implemented. Core methods
nsHTMLEditor::IncreaseFontSize and DecreaseFontSize need to be written.
Summary: Replace Font Size combobox with Increase and Decrease size buttons → [dogfood] Increase/Decrease font size need to be written
I see these new icons in the toolbar today but they don't seem to work.
This is a dogfood issue (need to be able to change the size of the text).
Change title to reflect latest comments by cmanske.

How do the new UI elements work if some of the selection is size 1 and some size
2?  Does it make them all size 2 or increment each part individually or make them
all size 3 or ???
Whiteboard: [PDT-]
Putting on PDT- radar.  Not needed for dogfood.
Blocks: 18951
Target Milestone: M12 → M13
Moving to M13
*** Bug 21755 has been marked as a duplicate of this bug. ***
Summary: [dogfood] Increase/Decrease font size need to be written → [feature] Increase/Decrease font size need to be written
Whiteboard: [PDT-]
Target Milestone: M13 → M14
Assignee: cmanske → jfrancis
Status: ASSIGNED → NEW
Target Milestone: M14 → M16
setting to m16 and reassigning to jfrancis
Status: NEW → ASSIGNED
Target Milestone: M16 → M14
moving to m14.  who needs sleep...

setting keyword
Keywords: beta1
Whiteboard: [PDT-]
Putting on PDT- radar for beta1.  Not critical to beta.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
fixed.  Sujay, it should be in todays build.  You dont need to wait a day.

it works, but we convert the indented text to blockquote style....

verified in 2/9 build.
Status: RESOLVED → VERIFIED
No longer blocks: 18951
I'm reopening this because it increase/decrease font size does not
work for partial text selection using 7/6 build.

1) enter some text
2) highlight half of the text
3) increase/decrease font size

nothing happens..

works if you select all of the text.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
fix in hand, very simple fix.
Status: REOPENED → ASSIGNED
Keywords: beta1 → nsbeta2
Whiteboard: [PDT-] → fix in hand
fixed
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → FIXED
verified in 7/25 build
Status: RESOLVED → VERIFIED
Build 2001091303
Win95 OSR-2 
Bug status is listed as fix but after highlighting text in
composer documents and using the keyboard shortcuts cntrl+= and cntrl+-
I noted the font size is still not being altered at all 
21755 also documents this occuring when attempting to resize with the
tool bar and that is also not fixed. 
Reproducible: Everytime

Isn't the "new" problem simply a dup of bug 99406?
no, it's yet a different problem.  I think it's either that the shortcut keys 
don't work on Windows or that someone else has defined a conflicting shortcut key 
so ours isn't getting triggered.  Regardless, please do NOT open up bugs that 
were fixed and verified more than 6 months ago.  I believe there *is* a bug filed 
on the issue raised above (look for "ctrl-+").
Component: Editor: Core → Editor: Composer
Using Ctrl + and Ctrl - work just fine to increase/decrease font size of both 
selected text or at the caret (Windows2000, build from 9/20/01.
Please be sure there are not <font size=...> tags in the text you are testing.

Regardless of current behavior, please take it to another bug; this bug is 
ancient.

Fixed/Verified Bugs older than 6 months should be left as such.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.