no keyboard (menu) access to insert-smiley feature

VERIFIED FIXED in mozilla1.1beta


18 years ago
11 years ago


(Reporter: jruderman, Assigned: vparthas)


({access, sec508})

Windows 98
access, sec508

Firefox Tracking Flags

(Not tracked)



(4 attachments, 3 obsolete attachments)



18 years ago
Most of the toolbar buttons in message compose have menu equivalents, but the 
insert-smiley feature doesn't.  This means there's no way to insert a graphical 
smiley using the keyboard :(

(Filing against Editor instead of Message Composition because I think this 
feature is also used in IM, and because bug 66213 is in the Editor component.)


18 years ago
Keywords: access

Comment 1

18 years ago
This should go to
Assignee: beppe → anatoliya

Comment 2

18 years ago
Created attachment 32665 [details] [diff] [review]
added smiley to editor

Comment 3

18 years ago
Please r/sr.

Comment 4

18 years ago
Anatoliy, this bug isn't requesting that smileys be added to Composer. I think 
it was already decided that we shouldn't be adding smileys to Composer.

This bug is about the fact that most of the items on MsgCompose's toolbar are 
also under the main menus ... smileys are only available via the toolbar. The 
user is also requesting that keyboard shortcuts for the various smileys be 

Comment 5

18 years ago
Anatoliy does not yet need r= or sr=.  He is working on a new fix which will add 
accessibility but not add the smiley feature to Composer.

This is really an accessibility bug and should be in Keyboard Navigation (not 

Jesse--are you suggesting that the insert smiley menu should be a hierarchical 
menu below the insert menu?
Component: Editor → Keyboard Navigation

Comment 6

18 years ago
No! We don't wan't the "Smiley" button on the toolbar.
Adding it as a submenu under "Insert" is acceptable.

Comment 7

18 years ago
Yes, there should be a submenu or menu item for inserting smilies in the Insert 
menu of the Message Compose window.  This shouldn't be necessary in a normal 
Composer window, since smilies are only supported in messages anyway.

Would using a submenu (as opposed to a menu item that brings up a dialog) only 
allow the text descriptions of the smilies to be shown, or do Mozilla's menus 
support large icons?

Comment 8

18 years ago
Keyboard access to all features is a requirement for Mozilla-based products to
be sold to the goverment.
Severity: minor → normal
Keywords: fcc508

Comment 9

18 years ago
does the reg require that people be able to trigger things the same exact way?

if users can insert smilies using the keyboard i don't quite understand why we 
need to let them activate a dropdown widget (although i wouldn't oppse support 
for tabbing through toolbars).

Comment 10

18 years ago
Well, it's a feature that you can choose the smiley based on a named emotion.
That feature is not available.

I suppose it could slide on the technicality though.

Comment 11

17 years ago
removing myself from the cc list
Created attachment 84118 [details] [diff] [review]
Add Smiley face menu after Insert/HTML in HTML message compose


17 years ago
Keywords: patch, review
Is there any Mozilla app which sends/saves these *graphical* smilies? If not, we
have an easy and intuitive keyboard access: Type ":-)".

Even if some Mozilla app sends graphical smilies (Mailnews surely does not), I
don't think that anybody limited to keyboard is keen on using them. Text-only
smilies are so commonplace and so much faster and more convient to type that I
don't think that anybody used to the keyboard will bother about graphical smilies.

In fact, I think don't understand why we have these graphical smilies at all.
Anyways, adding a submenu just so we can add key accelerators sounds quite silly
to me. As if we didn't have enough menus already.

As for government requirements, I'd argue that text-only smilies do it, for the
above reason.

Comment 14

17 years ago
Aaron--would you prefer Neil's patch or Ben's argument?  If you prefer Ben's
argument, we should resolve this bug as wontfix.

Comment 15

17 years ago
I think Ben's argument makes sense, except in IM (I know, this is not in
Mozilla). In IM we already have the menu, which contains a list of emotions that
you can pick from. In other words, if there is already a menu available for
mouse users, it should also be available from the keyboard. If there is not, we
can leave it alone.

On the other hand, can anyone name off the different ascii equivalents for the
different emotions we support? Are they actually listed anywhere?

Comment 16

17 years ago
aim 4.8.2646
:-) =-O :-$ :-\
:-( :-* :-! :'(
;-) >:o :-[ :-X
:-P 8-) O:-) :-D
procedure: load aim, click each smily, copy to nc4
as for giving friendly names ... um

the following is subject to analysis or negotation
) smile
( frown
; wink
P thpbbt
= startled
* smoking
> shocked
8 cool
$ scared
! surprised
[ toungue biting
0 angelic
\ disappointed
' crying
X silenced
D happy toungue

Comment 17

17 years ago
For IM or email it isn't suitable. Because it must be compatible with another 
IM clients (they are not Netscape or Mozilla).
So, we need to support 3-char sequences.

Comment 18

17 years ago
I've found that :-D is typically "Yay" and ":-/" is typically "worried"

In some communities, :-X or :-* is kiss, in others it's silence & smoking as

Comment 19

17 years ago
comment 17 is cryptic. what in the world was it in reference to?

my single character prefixes where for accesskeys for a menu which had the 
smilies listed previously.


17 years ago
Depends on: 145951

Comment 20

17 years ago
For Instant Messenger:,
"Can't access smiley menu via keyboard (Menus or shortcuts)".

Comment 21

17 years ago
*** Bug 155104 has been marked as a duplicate of this bug. ***

Comment 22

17 years ago
Re-assigning bug to varada
Assignee: anatoliya → varada
Component: Keyboard Navigation → Composition
Product: Browser → MailNews
Target Milestone: --- → mozilla1.1beta

Comment 23

17 years ago
Created attachment 91878 [details] [diff] [review]
Editor part of patch

Importing patch from 155104
Attachment #32665 - Attachment is obsolete: true
Attachment #84118 - Attachment is obsolete: true

Comment 24

17 years ago
Created attachment 91879 [details] [diff] [review]
MailCompose portion of patch

Mailcompose portion of patch
Comment on attachment 91878 [details] [diff] [review]
Editor part of patch

Attachment #91878 - Flags: review+
Comment on attachment 91879 [details] [diff] [review]
MailCompose portion of patch

missing the diff for the jar file (and maybe the .in file to) Appart that,
Attachment #91879 - Flags: review+

Comment 27

17 years ago
Created attachment 91881 [details] [diff] [review]
changes to

Changes to jar and there is no there.
Comment on attachment 91881 [details] [diff] [review]
changes to

Attachment #91881 - Flags: review+
Comment on attachment 91879 [details] [diff] [review]
MailCompose portion of patch

Where are your entities?
I don't see insertSmiley.label defined anywhere. Also while the
smileyNCmd.label entities have definitions in editorOverlay.dtd you are trying
to use messengercompose.dtd which does not define them.
Attachment #91879 - Flags: needs-work+

Comment 30

17 years ago
Created attachment 91962 [details] [diff] [review]
Changes to dtd files

Localization Changes for smiley menu items.

Comment 31

17 years ago
has anyone one the UE team looked at the new strings? cc'ing Jennifer.

Comment 32

17 years ago
Moving comments from DUP bug #155104.
------- Additional Comment #2 From 2002-07-01 10:27 -------

Add "_S_miley" to the Insert Menu below "Characters and Symbols".

Flyout smiley menu:

Comment 33

17 years ago
>Additional Comment #30 From 2002-07-19 09:44
>Created an attachment (id=91962)
>Changes to dtd files

Looks good.
Only comment, are you adding the tooltip:
+<!ENTITY SmileButton.tooltip "Choose smiley face">
If so, can it say "Insert a smiley face" instead?

Comment 34

17 years ago
Created attachment 95653 [details] [diff] [review]
Changes to dtd files v1.1

Changed the tooltip wording as per Jennifer's suggestions.
Attachment #91962 - Attachment is obsolete: true

Comment 35

17 years ago
Comment on attachment 95653 [details] [diff] [review]
Changes to dtd files v1.1

Attachment #95653 - Flags: superreview+

Comment 36

17 years ago
Marking Fixed.
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 37

16 years ago
QA Contact: sujay → gbush
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.