Hello, the edit-copy ID used by l10_client module in his form is already declared in form of contact module and that breaks the XHTML validation when these two forms are in the same page.

So i have made a little patch to change id of form elements to avoid this problem (this patch contains also the http://drupal.org/node/361147 patch which adds textgroup support)

CommentFileSizeAuthor
#3 ids-to-class-d7.patch7.67 KBGábor Hojtsy
#3 ids-to-class-d6.patch905 bytesGábor Hojtsy
#1 classify.patch6.26 KBGábor Hojtsy
l10n_client.module.patch5.82 KBsinasquax
l10n_client.js_.patch5.28 KBsinasquax
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Version: 6.x-1.7 » 7.x-1.x-dev
Status: Active » Patch (to be ported)
FileSize
classify.patch6.26 KB

Ok, based on similar changes in #616106: When another form contains same ids, client stops working, I've opted to use classes instead. The textgroup support was added in another recently committed patch, so that can be safely ignored here. This however needs porting to Drupal 7 (was just comitted to D6).

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

Should also add this little change (committing to D6 too):

-  $('#l10n-client #search-filter-clear').click(function() {
+  $('#l10n-client #l10n-client-search-filter-clear').click(function() {
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Patch (to be ported) » Fixed
FileSize
ids-to-class-d6.patch905 bytes
ids-to-class-d7.patch7.67 KB

Patch ported to D7 and found one more occurrence of #search-filter-clear in D6 too. Committing both.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.