954597 - Accessibility: Add a title to the chat log document in the conversation window

Status: RESOLVED FIXED

(Instantbird Graveyard :: Conversation, defect)

Description

[Instantbot]

*** Original post on bio 1165 by Peter Vagner <pvdeejay AT gmail.com> at 2011-11-13 20:00:00 UTC ***

Instantbird chat window has at least the following focusable controls.
- input box - an edit field where user types messages to sent to his chat peers.
- a tab list - where user can switch individual conversations.
- A chat log - a read only control where incoming and outgoing messages are displayed.
There is a minor accessibility issue related to this chat log control.
There is a document rendered inside which has no title. If there is no title gecko generates the title out of the document url.
It's too verbose and ugly to be presented with such a horrible title each time focusing the chat log control.
The automatically generated title reads for example:
wyciwyg://1/chrome://instantbird/content/instantbird.xul 
I have simply added a <title>Chat log</title> into the document markup inside the initHTMLDocument in the file modules/imThemes.jsm.
Of course localization should be taken into account while fixing this properly.

Comment 1

[Patrick Cloke [:clokep]]

*** Original post on bio 1165 at 2012-02-16 21:48:26 UTC ***

Thanks for filing this! Sorry no one has responded yet. We do need to add this and I'm marking it as new.

Comment 2

[Benedikt Pfeifer]

Attachment: Patch v1

*** Original post on bio 1165 as attmnt 1240 at 2012-03-10 00:01:00 UTC ***

I inserted a title-tag at the given place and used the conversation title as value. I guess that's better than a generic "Chat log" title?

Comment 3

[Patrick Cloke [:clokep]]

Comment on attachment 8352993 [details] [diff] [review]
Patch v1

*** Original change on bio 1165 attmnt 1240 at 2012-03-10 00:23:47 UTC ***

This looks fine, I'm not sure I love the comment...but it seems correct.

Comment 4

[Instantbot]

*** Original post on bio 1165 by Peter Vagner <pvdeejay AT gmail.com> at 2012-03-10 07:17:11 UTC ***

I think this is perfect.
Thanks verry much!

Comment 5

[Instantbot]

*** Original post on bio 1165 by Peter SZTAN <sztanpet AT gmail.com> at 2012-03-11 23:45:44 UTC ***

this probably should be separate bug, but the dialogue asking for the master password should probably also include the name of the application at least

Comment 6

[Patrick Cloke [:clokep]]

*** Original post on bio 1165 at 2012-03-12 23:52:30 UTC ***

Checked in as http://hg.instantbird.org/instantbird/rev/c15791758442 thanks Peter for the report and Mic for the patch!