Closed Bug 309081 Opened 15 years ago Closed 13 years ago

Sync xpfe autocomplete attributes with toolkit ones (sync case)

Categories

(Toolkit :: Autocomplete, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9

People

(Reporter: mcsmurf, Assigned: standard8)

References

Details

Attachments

(2 files)

There is the property named forceComplete in both toolkit/ and xpfe/
autocomplete to force autocomplete. But the toolkit version looks like this:
<property name="forceComplete"
          onset="this.setAttribute('forcecomplete', val); return val;"
          onget="return this.getAttribute('forcecomplete') == 'true';"/>

And the xpfe version like this:
<property name="forceComplete"
          onset="this.setAttribute('forceComplete', val); return val;
          onget="return this.getAttribute('forceComplete') == 'true';"/>

I think we should make work for developers and upcoming transitions easier, we
should just agree on one of the two versions here.
Blocks: 360648
That would be helpful... there are a few other attributes which could do with being synchronised although I don't know how many of them Thunderbird use:

disableAutocomplete (property name mismatch!)
minResultsForPopup	ignoreBlurWhileSearching
showCommmentColumn	tabScrolling

Plus some attributes which toolkit doesn't have, but we might want to lowercase for consistency:

highlightNonMatches	autoFillAfterMatch	autoFill
Assignee: nobody → bugzilla
Summary: Sync forcecomplete with xpfe autocomplete → Sync xpfe autocomplete attributes with toolkit ones (sync case)
This is the patch that will do the changes to the lower case for all the attributes mentioned in comment 2. I'll request review on the mail/ changes once I know Neil's happy with the rest of them.

I'm doing a separate patch for Calendar, as that will need to work with the 1.8 branch as well as trunk.
Attachment #320869 - Flags: superreview?(neil)
Attached patch Calendar FixSplinter Review
Comment on attachment 320869 [details] [diff] [review]
Fix xpfe autocomplete and mailnews and mail

>+                 defaultSearchEngine="true" tabscrolling="true"
Remind me to file a bug on fixing urlbarBindings.xml for consistency.
Attachment #320869 - Flags: superreview?(neil) → superreview+
Attachment #320869 - Flags: review?(mkmelin+mozilla)
Attachment #320870 - Flags: review?(daniel.boelzle)
Comment on attachment 320869 [details] [diff] [review]
Fix xpfe autocomplete and mailnews and mail

Looks fine, r=mkmelin
One nit: the change around _awSetAutoComplete - would be good to fix the overlong line
Attachment #320869 - Flags: review?(mkmelin+mozilla) → review+
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9
Comment on attachment 320870 [details] [diff] [review]
Calendar Fix

>+            <!-- Where attributes are repeated, this allows compatibility with
>+                 the Mozilla 1.8 branch -->
Could you please change "Mozilla 1.8 branch" to "MOZILLA_1_8_BRANCH", because we will presumably grep for that string when cleaning up the branch specifics in the future.

otherwise looks good; r=dbo

Please cross-commit to both branch and trunk.
Attachment #320870 - Flags: review?(daniel.boelzle) → review+
Calendar patch checked in on branch and trunk:

Checking in calendar/prototypes/wcap/sun-calendar-event-dialog-attendees.xml;
/cvsroot/mozilla/calendar/prototypes/wcap/sun-calendar-event-dialog-attendees.xml,v  <--  sun-calendar-event-dialog-attendees.xml
new revision: 1.42; previous revision: 1.41
Checking in calendar/prototypes/wcap/sun-calendar-event-dialog-attendees.xml;
/cvsroot/mozilla/calendar/prototypes/wcap/sun-calendar-event-dialog-attendees.xml,v  <--  sun-calendar-event-dialog-attendees.xml
new revision: 1.1.2.42; previous revision: 1.1.2.41

Main patch checked in on trunk:
Checking in mail/components/addrbook/content/abEditListDialog.xul;
/cvsroot/mozilla/mail/components/addrbook/content/abEditListDialog.xul,v  <--  abEditListDialog.xul
new revision: 1.10; previous revision: 1.9
done
Checking in mail/components/addrbook/content/abMailListDialog.xul;
/cvsroot/mozilla/mail/components/addrbook/content/abMailListDialog.xul,v  <--  abMailListDialog.xul
new revision: 1.10; previous revision: 1.9
done
Checking in mail/components/compose/content/addressingWidgetOverlay.js;
/cvsroot/mozilla/mail/components/compose/content/addressingWidgetOverlay.js,v  <--  addressingWidgetOverlay.js
new revision: 1.16; previous revision: 1.15
done
Checking in mail/components/compose/content/messengercompose.xul;
/cvsroot/mozilla/mail/components/compose/content/messengercompose.xul,v  <--  messengercompose.xul
new revision: 1.122; previous revision: 1.121
done
Checking in mailnews/addrbook/resources/content/abListOverlay.xul;
/cvsroot/mozilla/mailnews/addrbook/resources/content/abListOverlay.xul,v  <--  abListOverlay.xul
new revision: 1.24; previous revision: 1.23
done
Checking in mailnews/compose/resources/content/addressingWidgetOverlay.js;
/cvsroot/mozilla/mailnews/compose/resources/content/addressingWidgetOverlay.js,v  <--  addressingWidgetOverlay.js
new revision: 1.98; previous revision: 1.97
done
Checking in mailnews/compose/resources/content/addressingWidgetOverlay.xul;
/cvsroot/mozilla/mailnews/compose/resources/content/addressingWidgetOverlay.xul,v  <--  addressingWidgetOverlay.xul
new revision: 1.70; previous revision: 1.69
done
Checking in suite/browser/navigator.xul;
/cvsroot/mozilla/suite/browser/navigator.xul,v  <--  navigator.xul
new revision: 1.460; previous revision: 1.459
done
Checking in xpfe/components/autocomplete/resources/content/autocomplete.xml;
/cvsroot/mozilla/xpfe/components/autocomplete/resources/content/autocomplete.xml,v  <--  autocomplete.xml
new revision: 1.162; previous revision: 1.161
done

Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/f4811ba7c780
Sync xpfe autocomplete attributes with toolkit ones (sync case). r=mkmelin,sr=Neil
You need to log in before you can comment on or make changes to this bug.