327998 - move accesskey label into xforms-xhtml.xml

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[alexander :surkov (:asurkov)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051212 Firefox/1.6a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051212 Firefox/1.6a1

move accesskey label into xforms-xhtml.xml

Reproducible: Always

Comment 1

[alexander :surkov (:asurkov)]

Attachment: testcase

Comment 2

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 3

[alexander :surkov (:asurkov)]

Attachment: update

Comment 4

[Allan Beaufour]

Checked into trunk

Comment 5

[alexander :surkov (:asurkov)]

Attachment: accesskey issue

Now label-accesskey widget is based on label widget. The patch fixes non binded label-accesskey (invalid behaviour on second refresh).

Comment 6

[aaronr]

*** Bug 329027 has been marked as a duplicate of this bug. ***

Comment 7

[aaronr]

Attachment: calc testcase

Looks like the 'accesskey issue' patch fixes my testcase on 329027, but then it rebreaks the calculator sample (which I am attaching here) that your previous patch fixes.  So when this patch is finished and ready for checkin, please verify that you can switch to SVG and back to 'normal' on this testcase and still pass the accesskey part of the other attached testcase.  Thanks.

Comment 8

[alexander :surkov (:asurkov)]

(In reply to comment #7)
> Created an attachment (id=213676) [edit]
> calc testcase
> 
> Looks like the 'accesskey issue' patch fixes my testcase on 329027, but then it
> rebreaks the calculator sample (which I am attaching here) that your previous
> patch fixes.  So when this patch is finished and ready for checkin, please
> verify that you can switch to SVG and back to 'normal' on this testcase and
> still pass the accesskey part of the other attached testcase.  Thanks.
> 

The problem is like problem described in bug 313857. html:span element of xforms:label has zero width.

Comment 9

[alexander :surkov (:asurkov)]

Attachment: patch (calctest works)

Comment 10

[Doron Rosenberg (IBM)]

Comment on attachment 213880 [details] [diff] [review]
patch (calctest works)

>diff -uNra mozilla.orig/extensions/xforms/resources/content/xforms.css mozilla/extensions/xforms/resources/content/xforms.css
>--- mozilla.orig/extensions/xforms/resources/content/xforms.css	2006-03-01 18:34:30.000000000 +0800
>+++ mozilla/extensions/xforms/resources/content/xforms.css	2006-03-03 18:32:39.000000000 +0800
>@@ -280,14 +280,14 @@
>   color: GrayText !important;
> }
> 
>-html|*:root input > label,
>-html|*:root secret > label,
>-html|*:root textarea > label,
>-html|*:root trigger > label,
>-html|*:root submit > label,
>-html|*:root select > label,
>-html|*:root select1 > label,
>-html|*:root upload > label {
>+html|*:root input[accesskey] > label,
>+html|*:root secret[accesskey] > label,
>+html|*:root textarea[accesskey] > label,
>+html|*:root trigger[accesskey] > label,
>+html|*:root submit[accesskey] > label,
>+html|*:root select[accesskey] > label,
>+html|*:root select1[accesskey] > label,
>+html|*:root upload[accesskey] > label {
>   -moz-binding: url('chrome://xforms/content/xforms-xhtml.xml#xformswidget-label-accesskey');
> }

Do we need this root stuff?

Comment 11

[alexander :surkov (:asurkov)]

(In reply to comment #10)

> 
> Do we need this root stuff?
> 

Yes, I guess we need because f.x. we should have other realization of accesskey label control for xul (The one reason is I guess it's not proper way to underline in xul like it done for xhtml accesskey-label).

Comment 12

[Allan Beaufour]

Comment on attachment 213880 [details] [diff] [review]
patch (calctest works)

Is this patch still needed? The testcases seem to work on standard trunk build for me...

Comment 13

[alexander :surkov (:asurkov)]

(In reply to comment #12)
> (From update of attachment 213880 [details] [diff] [review] [edit])
> Is this patch still needed? The testcases seem to work on standard trunk build
> for me...
> 

Yes, it's needed. The bug 329027 shows the problem. When label-accesskey gets second refresh then it will have wrong look.

Comment 14

[Allan Beaufour]

(In reply to comment #9)
> Created an attachment (id=213880) [edit]
> patch (calctest works)

Checked into trunk