The default bug view has changed. See this FAQ.

Cursor at end of text of import calendar list pop-up/popup/window/dialog

RESOLVED FIXED in 2.1

Status

Calendar
Import and Export
--
minor
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: aryx, Assigned: aryx)

Tracking

Lightning 1.0b2
x86
Windows XP

Details

Attachments

(2 attachments, 1 obsolete attachment)

Created attachment 468124 [details]
calendar import popup

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.8) Gecko/20100802 Lightning/1.0b2 Thunderbird/3.1.2

I can see a cursor cursor at the end of the import calendar list pop-up/popup/window/dialog.
Version: Trunk → Lightning 1.0b2
Fallen, are you okay with a patch replacing the textbox with a description with a text child node in chooseCalendarDialog.xul? This works for me in a test:

<description id="prompt"></description>

and

document.getElementById("prompt").textContent = window.arguments[0].promptText;
Looks like the review comment from bug 323085 comment 17 was never actually taken care of ;-)

Yes, that sounds good. While you are here, could you also add some accessibility features, I think it was:

<description id="prompt" for="id-of-listbox"/>

Please doublecheck.
Created attachment 672053 [details] [diff] [review]
patch: replace textbox with wrapping label in chooseCalendarDialog.xul, v1

For XUL label, it's control="id", not for="id" like in HTML.
Assignee: nobody → archaeopteryx
Status: NEW → ASSIGNED
Attachment #672053 - Flags: review?(philipp)
Comment on attachment 672053 [details] [diff] [review]
patch: replace textbox with wrapping label in chooseCalendarDialog.xul, v1

Review of attachment 672053 [details] [diff] [review]:
-----------------------------------------------------------------

r=philipp with the following nit fixed:

::: calendar/base/content/dialogs/chooseCalendarDialog.xul
@@ +80,5 @@
>          }
>      ]]></script>
>  
>      <vbox id="dialog-box" flex="1">
> +        <label id="prompt" control="calendar-list"></label>

To align with the rest, you could use a short close tag.
Attachment #672053 - Flags: review?(philipp) → review+
Created attachment 682592 [details] [diff] [review]
patch for checkin: replace textbox with label, v2
Attachment #672053 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/890a72806339
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.9
Target Milestone: 1.9 → 2.1
You need to log in before you can comment on or make changes to this bug.