- ๐ณ๐ฑNetherlands Johan den Hollander
The #87 patch applies cleanly and does what it has to do. The inactive tab gets opened and shows the required field that was left empty.
The last submitted patch, 31: field_group-HTML5_validation_prevents_submission_in_tabs-2969051-31.patch, failed testing. View results โ
The last submitted patch, 40: 2969051-40.patch, failed testing. View results โ
I'm finding the #87 patch works, but I am still seeing the error in the console. Since an earlier patch I've used (#71) got rid of the console error, I was not expecting to see it.
- ๐ช๐จEcuador andres.torres
Hi All,
I'm reopening this since it still an issue when using the Multistep option. We have a contentype with 3 multi-groups under this scenario but unfortunately after upgrading to x.3.4 they stop working on the Admin side (not even user 1 can create content). I've applied the patch on #87 successfully but the issue persists, I have also disabled the HTML5 validation on ALL forms $form['#attributes']['novalidate'] = 'novalidate'; on a custom module. There are no permission implemented, as i mentioned not even user 1 can go through the second step (we have 3 steps using field group ) Works OK on the front end theme. Gin works ok for other contentypes without Field Group. Dont see any Javascript errors on console or any other php errors either.
- Status changed to Needs review
almost 2 years ago 2:37pm 16 March 2023 - Status changed to Needs work
almost 2 years ago 2:49pm 16 March 2023 - ๐ฎ๐ณIndia gaurav_manerkar Vasco Da Gama, Goa
Patch #87 doesn't work when the form is rendered via ajax.
- ๐ฎ๐ณIndia gaurav_manerkar Vasco Da Gama, Goa
I have updated sector and hv re-rolled patch #87.
- last update
over 1 year ago 18 pass - last update
over 1 year ago 18 pass - Status changed to Needs review
over 1 year ago 2:48pm 24 April 2023 - ๐ฆ๐บAustralia mstrelan
FWIW a similar problem was fixed in core for vertical tabs. See ๐ Correct vertical tab does not focus on form validation Fixed
- Status changed to Needs work
over 1 year ago 4:12pm 5 July 2023 - ๐บ๐ธUnited States kevinquillen
Latest patch seems to have no effect in Gin and Field Group in Drupal 10.
- ๐จ๐ญSwitzerland ayalon
The latest patch works for me with tabs on Gin in Drupal 9. Thanks a lot for the huge improvement.
- Status changed to RTBC
over 1 year ago 12:59pm 12 September 2023 - ๐บ๐ธUnited States dpagini
The last 2 comments are saying the opposite... could we maybe get more elaboration on #103? I'm using this patch with Claro and it is resolving this issue, so I'm going to put to RTBC, but with the caveat that I'm not testing in Gin and 103/104 are saying it's working/it's not.
- Status changed to Needs work
about 1 year ago 5:55am 9 October 2023 - ๐ฉ๐ชGermany Anybody Porta Westfalica
- First commit to issue fork.
- Merge request !40Ensure respective tab is opened when html validation for a field prevents form submission โ (Open) created by sahal_va
- last update
about 1 year ago 24 pass - last update
about 1 year ago 24 pass - Status changed to Needs review
about 1 year ago 3:48am 11 November 2023 - ๐บ๐ธUnited States dianacastillo Miami
works for me in drupal 9.5 will this be in the latest version of the module soon?
- ๐ฌ๐งUnited Kingdom aaron.ferris
Latest patch works well for me, Drupal 9.5.9 and Field Group 3.4.0.
- ๐ณ๐ฑNetherlands helpermedia
Patch #100 works also well for me, Field Group 3.4 and Drupal 10.1.6 with Claro theme.
- last update
12 months ago 17 pass, 2 fail - First commit to issue fork.
- last update
12 months ago 17 pass, 2 fail - ๐ง๐ทBrazil brandonlira
The last patch #100 didn't work for me because https://www.drupal.org/project/drupal/issues/2722319 ๐ s using Text Editor always fail HTML5 validation when "required" is added via #states Needs work CKEditor removed the "required" attribute from the textarea during startup. So I adjusted the patch with these changes.
The last submitted patch, 114: 2969051-100.patch, failed testing. View results โ
- last update
12 months ago 17 pass, 2 fail - ๐ง๐ทBrazil brandonlira
The last patch #100 didn't work for me because https://www.drupal.org/project/drupal/issues/2722319 ๐ s using Text Editor always fail HTML5 validation when "required" is added via #states Needs work CKEditor removed the "required" attribute from the textarea during startup. So I adjusted the patch with these changes.
The last submitted patch, 116: 2969051-100.patch, failed testing. View results โ
- Status changed to Needs work
12 months ago 1:51pm 3 January 2024 - ๐ซ๐ทFrance dydave
Hi everyone,
Thanks a lot for raising this issue and contributing a patch, it's greatly appreciated !
I would like to confirm the patch from #116 applied properly with composer and seems to have fixed the issue in my case with the Gin admin theme and horizontal tabs (field_group), with the following versions:
- drupal/core: 10.2.0
- drupal/field_group: 3.4.0
- drupal/gin: 3.0.0-rc8
Client side validation now seems to be working properly by changing to the tab with the required field and focusing the right element.Thanks again very much for your help with this issue.
- ๐ซ๐ทFrance dydave
Correction:
After doing a little bit more testing, I've been having the following JS error with the patch from #116 after saving a content form:Uncaught TypeError: Cannot read properties of null (reading 'classList') tabs.js line 119 at Drupal.FieldGroup.fieldIsValid (tabs.js?119:34)
Which seems to correspond to the following line of code:
https://git.drupalcode.org/project/field_group/-/merge_requests/40/diffs...Added above at #113:
@brandonlira: Not sure exactly what use case that should cover but there might be a little bit more testing to be done, with perhaps a few more cases/checks to be added in the JS code.Reverted to using patch from #100 which fixed my validation issue without prompting this JS error on form save.
Thanks again everyone for the great help!
- ๐ฌ๐งUnited Kingdom MrDaleSmith
I have a site where the node aded form for a content type has every element inside a collapsed fieldset: applying the patches in #100 and #116 still generated JS errors that silently prevented form submission when I trry to submit the form without filling anything.
- ๐ซ๐ทFrance fwust Strasbourg
#116 worked for me in Gin admin theme with horizontal tabs. But when i add a paragraph with field groups, it doesn't work anymore
- ๐ซ๐ทFrance MacSim
The patch provided in #100 and MR!40 does not fully work for me.
I use horizontal tabs with a Claro 10.2.5.
If there are required fields in another tab that haven't been filled out, the JS takes me to one of those other tabs when I add a paragraph to the current tab. - ๐ซ๐ทFrance MacSim
Small update of patch provided in #100 ๐ HTML5 Validation Prevents Submission in Tabs Postponed: needs info in order not to break the behavior of adding a paragraph to a tab.
I tried
'.form-submit:not([formnovalidate,.field-add-more-submit])'
but it was not a valid selector (at least on firefox) ; so I just chained another :not() =>'.form-submit:not([formnovalidate]):not(.field-add-more-submit)'
and I am wondering if it's a good idea to have an array declaration in the first "not" since it's not well supported by all browsers yet. - Status changed to Needs review
8 months ago 12:22pm 10 April 2024 - First commit to issue fork.
- Merge request !51Issue #2969051 by gaurav_manerkar, julien, rp7: HTML5 Validation Prevents Submission in Tabs โ (Open) created by jorgik
- Status changed to RTBC
6 months ago 6:14am 19 June 2024 - ๐ฆ๐บAustralia jannakha Brisbane!
#124 applied and fixes the issue!
Merci @MacSim !any other testers?
- ๐ณ๐ดNorway steinmb
To help getting this committed have I applied this into one of our production sites. I'll also make sure to send testers in there and take if for a spin.
- First commit to issue fork.
- ๐ฌ๐งUnited Kingdom scott_euser
scott_euser โ changed the visibility of the branch 8.x-3.x to hidden.
- ๐ฌ๐งUnited Kingdom scott_euser
scott_euser โ changed the visibility of the branch 2969051-html5-validation-prevents to hidden.
- ๐ฌ๐งUnited Kingdom scott_euser
Bit of a mess between patches & merge requests & outdated merge request fork.
Now the remaining MR is actually merge-able and contains identical content to the patch in #124. As such, marketing this as RTBC as I haven't actually made any change - just did a bunch of issue clean-up - and testing this it works as expected.
For future travellers please follow https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... โ and download the patches locally from the MR to your project as per the recommendations/instructions rather than adding more patches here and having patches and MRs diverge again. Ie, let's keep us all moving forward on the same track and make life easier for the maintainers of this module to review & merge. Thank you!
- ๐ฌ๐งUnited Kingdom scott_euser
Okay I believe credit list is also now up to date up; tried to also help maintainers by reviewing all people who have made contributions matching/exceeding the guidelines (ie, not just 'it works' type comments, meaningful reviews, code contributions, etc)
- Status changed to Needs review
6 months ago 11:30am 25 June 2024 - ๐ฌ๐งUnited Kingdom scott_euser
Okay I have made changes now, sorted all eslint errors, fixed typos.
The phpstan issues are unrelated to this MR, they are new to the module and therefore raised separately.
To help progress this issue if someone could please:
- Review and test ๐ Fix deprecated extending of render element Fixed to RTBC it so tests here can go all green (including phpstan)
- Re-review this consider the changes I made to eslint/typos to get this back to RTBC
Then we'll hopefully all get the satisfaction of getting a 6 year old issue over the line :)
- Status changed to RTBC
6 months ago 10:45am 26 June 2024 - ๐ฆ๐บAustralia jannakha Brisbane!
tested MR!57 on D10.3, php 8.3, on 3.x-dev@dev
looks good!Thanks for fixing the patches/MRs!
- ๐ท๐ดRomania andreic
I've just tested on D10.2.6, php 8.3, on 3.x-dev@dev and it doesn't get applied.
I attached 2 screenshots: first when applying the patch through composer.json and the second manually. - ๐ณ๐ฑNetherlands roaldnel
I found the same as andreic, the code updates do not apply on 3.x-dev@dev or 8.x-3.6 for us.
- ๐น๐ทTurkey emircan erkul Turkey
Emircan Erkul โ made their first commit to this issueโs fork.
- First commit to issue fork.
- ๐บ๐ฆUkraine Taran2L Lviv
@Emircan Erkul there is no point of creating a new MR (unless the current one cannot be updated easily)
- ๐บ๐ฆUkraine Taran2L Lviv
Taran2L โ changed the visibility of the branch 2969051--patch-rollout--8.x-3.6 to hidden.
- ๐ฎ๐ณIndia nitesh624 Ranchi, India
for me also patch from #99 broken after 3.6 upgrade
- ๐ท๐ดRomania andreic
I've just used the latest version of the module (3.6.0 from 3rd of Aug) and now I get for some horizontal tabs a double star (**) for tabs that are required and one start (*) for those that are NOT required.
Also, the main problem still stands: when I fill in the fields in the first tab and click Save at the bottom, nothing happens. It should open the next horizontal tab and highlight the next field that is required to complete, right?
- Status changed to Needs work
4 months ago 8:05am 8 August 2024 - ๐ฉ๐ชGermany spuky
i also see the double stars that Andreic is seeing after applying mr57 against 3.6 so setting to needs work..
- ๐ท๐ดRomania andreic
The patch applied successfully and it also fixed the issue.
But what is going on with the double start on vertical/horizontal tabs? I would add a screenshot but I don't have that option anymore for some reason. - ๐ท๐ดRomania andreic
Never mind, I found the patch for that in https://www.drupal.org/project/field_group/issues/3395375 ๐ Duplicated required marks in field tabs with GIn admin theme RTBC .
This issue can be closed now. - ๐ฉ๐ชGermany spuky
This issue can be closed now.
why should the issue be closed... it was not about the 2 Stars...
but about "HTML5 Validation Prevents Submission in Tabs" the thing to do would be to update the mr to match patch 148 and set the issue back to needs review or RTBC - ๐ท๐ดRomania andreic
Right, closed as in resolved further, meaning MR and RTBC.
Sorry, I wrote that in a hurry. - ๐ฌ๐งUnited Kingdom aaron.ferris
I looked to get the latest patch into the MR. Im seeing indentation changes (some funky indentation going on), but I can't see any functional differences compared to the MR, what am I missing? Is the patch meant to be introducing something different vs the MR?
- ๐ฉ๐ชGermany Anybody Porta Westfalica
I'd be happy to merge this, once someone resolves this finally!
- First commit to issue fork.
- Status changed to RTBC
3 months ago 4:24pm 6 September 2024 - ๐บ๐ฆUkraine Taran2L Lviv
@anybody, the MR is ready, PHPStan issue in unrelated deprecation, double asterisk in another issue, this is RTBC as far as I can tell
- ๐ฌ๐งUnited Kingdom scott_euser
Taran, thanks for confirming! For ease of maintainers review reference for double asterisk issue is ๐ Duplicated required marks in field tabs with GIn admin theme RTBC I believe.
- ๐ฉ๐ชGermany Anybody Porta Westfalica
Thank you all! Let's finall merge this into 4.x as RTBC has been confirmed several times and tests are green.
Please try 4.x in dev environments to ensure a brighter future :) - ๐ฉ๐ชGermany Anybody Porta Westfalica
anybody โ changed the visibility of the branch 8.x-3.x--pristine to hidden.
- Status changed to Needs work
3 months ago 9:51am 8 September 2024 - ๐ฉ๐ชGermany Anybody Porta Westfalica
A test is failing now, someone please needs to take a look. Thanks!
- Assigned to Grevil
- Issue was unassigned.
- Status changed to Postponed: needs info
3 months ago 10:09am 9 September 2024 - ๐ฉ๐ชGermany Grevil
Just talked with @Anybody internally, and we'll postpone this issue for now. We'll fix ๐ Ensure visibility of invalid fields (JS) RTBC first, which solves a quite similar (maybe even identical) issue and then people can give feedback here if anything else has to be fixed here!
- ๐ณ๐ดNorway steinmb
Issue ๐ Ensure visibility of invalid fields (JS) RTBC is now fixed in 4.x. and there is follow up issue from that that might be useful to keep an eye on ๐ Multiple required fields on multiple tabs do no trigger frontend validation Active . Not sure if we should postpone this issue on that. Changing to NW.
- ๐ฉ๐ชGermany Anybody Porta Westfalica
Postponed on ๐ Multiple required fields on multiple tabs do no trigger frontend validation Active as of #167. Makes sense for now.
- ๐บ๐ธUnited States dianacastillo Miami
which of the attached patches works with the 8.x-3.6 code.
- Status changed to Postponed
21 days ago 7:56pm 27 November 2024 - ๐ซ๐ทFrance rbrissaud
I fixed the setGroupWithfocus function call to setGroupWithFocus (camel case missing on focus)