Format submenus do not indicate current style of selection




19 years ago
18 years ago


(Reporter: ftang, Assigned: Brade)




Firefox Tracking Flags

(Not tracked)


(Whiteboard: [rtm++]Patch attached)


(6 attachments)



19 years ago
This is using Window commercial Netscape 6 bulid 2000092408
on Window NT4

repreduce procedure
1. launch Netscape6
2. Select "Task:composer" to open a new page.
3. type "abcdefg"
4. select "abc"
5. select "Format:Text Style:Bold"
6. the text "abc" display as bold
7. select "Format:Text Style" menu
expect result-
the menu item "Bold" show a check mark or some other indication to tell the 
user the current select text is in "Bold"

actual result-
no indication to tell user what the current state is.
"Format:Paragraph", "Format:List" and "Format:Align" menu do indicate the 
current state. 
"Format:Font" (as 54316) and "Format:Size" (as 54315)does not do this neither. 
Will report 
seperate bug for them.

Comment 1

19 years ago
This problem can be reproduce on 20000927 Win/Linux build. But it is ok on Mac

Comment 2

19 years ago
futuring, not sure who the owner of this is.

Target Milestone: --- → Future

Comment 3

19 years ago
there should be checkmark next to the style of the current selection, i thought 
that that used to be the case
Assignee: beppe → cmanske
Keywords: correctness, regression

Comment 4

19 years ago
This one is trivial. low risk/high visibility.
The menuitems were missing the 'type="checkbox"' attribute, which is needed
in order to have the checkmark visible.
Nominating for rtm fix.
Keywords: rtm
Target Milestone: Future → M19

Comment 5

19 years ago
Created attachment 15761 [details] [diff] [review]
Fix for bug: simply add 'type="checkbox"' on menuitems

Comment 6

19 years ago
Created attachment 15867 [details] [diff] [review]
Fix for missing feedback in all Format submenus (fixes 54315, 54316, and 54317)

Comment 7

19 years ago
If bugs 54315 and 54316 are also approved or merged with this bug to cover
all missing submenu feedback issues, use the 9/29 patch, which fixes all 3
Patch has been sent to Composer group for more testing and reviews.

Comment 8

19 years ago
Created attachment 15880 [details]
Test cases for Font face, size and text styles user feedback in Format submenus

Comment 9

19 years ago
The attached test file works for bugs 54315 and 54316 as well.
Handing off to Kathy as I'm starting my sabatical.
Assignee: cmanske → brade

Comment 10

19 years ago
the user needs to be able to determine the style information from the menu, ther 
e is no text property dialog, consequently there is no workaround, this is a 
highly visible function.

Kathy, please include the required information per the rtm checkin rules
Priority: P3 → P2
Whiteboard: [rtm+ NEED INFO]Patch attached

Comment 11

19 years ago
PDT agrees [rtm need info] until code reviews are done
Whiteboard: [rtm+ NEED INFO]Patch attached → [rtm NEED INFO]Patch attached
Keywords: mailtrack

Comment 12

19 years ago
Created attachment 16236 [details] [diff] [review]
patch as before but with additional fixes for Macintosh

Comment 13

19 years ago
Created attachment 16237 [details] [diff] [review]
JS patch (part 2 of 2)

Comment 14

19 years ago
I have attached new patches (2 separate attachments) because Charley's fixes
didn't work on Macintosh.  Apparently all radio menu items need to specify a
name (at least that is true on Macintosh).

Because I can't review my additional changes, I need reviews from both kin and
sfraser (one as sr and one as module owner).

There is no impact to i18n/l10n.
This is a very visible bug that seriously impacts editing in both Composer and
mail compose (the user has no feedback or incorrect feedback).
Severity: normal → major
OS: Windows NT → All
Hardware: PC → All

Comment 15

19 years ago
correcting summary to agree with the real problems addressed in this bug
Kin did a review and we agreed to one small change to the JS patch (part 2 of 2)
submitted 10/05/00 10:51.  I will attach the new diff for completeness

Kin--please add your sr= comment
Summary: Format:Text Style menu do not indicate the current style of the selected text → Format submenus do not indicate current style of selection
Whiteboard: [rtm NEED INFO]Patch attached → [rtm+]Patch attached

Comment 16

19 years ago
Created attachment 16326 [details] [diff] [review]
replacement JS patch (part 2 of 2)

Comment 17

19 years ago
*** Bug 54315 has been marked as a duplicate of this bug. ***

Comment 18

19 years ago
*** Bug 54316 has been marked as a duplicate of this bug. ***

Comment 19

19 years ago

sfraser said he wanted to take a look at the new diffs which brade just posted 
(10/06/00 06:01.

Comment 20

19 years ago

Comment 21

19 years ago

Comment 22

19 years ago
note: the correct patches for this bug are dated:
  10/05/00 10:50  and 10/06/00 06:01

I'll send this off to pdt.
Keywords: patch

Comment 23

19 years ago
Whiteboard: [rtm+]Patch attached → [rtm++]Patch attached

Comment 24

19 years ago
Fix checked into branch before today's builds.

Comment 25

19 years ago
fix now checked into trunk; resolving bug
Last Resolved: 19 years ago
Resolution: --- → FIXED

Comment 26

19 years ago
verified in 10/10 build.
You need to log in before you can comment on or make changes to this bug.