Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Pages: 1
r3819 forms nested in fieldset
The reshuffle in r3819 introduced a weirdo bug: on the image edit pane and the file edit pane forms are nested inside fieldset. That is invalid html. It should be the other way around !
Example (seen on the edit file pane):
<fieldset class="replace-file">
<legend>
Replace file <a rel="help" ….</legend>
<form class="upload-form" method="post" enctype="multipart/form-data" action="index.php">...
<p class="edit-file-replace"> <input type="file" value="" name="thefile" id="file_replace"> <input type="submit" value="Upload"></p>
<input type="hidden" value="48304ed1c15b6e5d13a61157c18de8ee" name="_txp_token">
</form>
</fieldset>(some code snipped for brevity)
Expected:
<form>
<fieldset><legend>
<input type=hidden ......>
etcWhere is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: r3819 forms nested in fieldset
phiw13 wrote:
formsare nested insidefieldset. That is invalid html. It should be the other way around !
I’m sure you’re right, but it beats me why the validator never complained when I ran it through each page. Weird.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: r3819 forms nested in fieldset
Beside validating (or not), it’s definitely an anti-pattern, imo.
Offline
Re: r3819 forms nested in fieldset
Philippe is correct, fieldset tag needs to go inside form tag in 4 instances (1 instance on file edit page, and 3 instances on image edit page).
Offline
Re: r3819 forms nested in fieldset
Rats. The <form> tags are generated by a function, and sometimes the forms do need fieldsets, and other times they don’t.
That’s gonna make for some ugly code, or a new wrapper function. The latter is the lesser of two weevils I guess.
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: r3819 forms nested in fieldset
Bloke wrote:
I’m sure you’re right, but it beats me why the validator never complained when I ran it through each page. Weird.
Weird indeed. But it doesn’t make sense. fieldset is a form control. But given the weasel wording that the htmls 5 spec is becoming, maybe the validator is correct.
http://dev.w3.org/html5/spec/single-page.html#forms
Where is that emoji for a solar powered submarine when you need it ?
Sand space – admin theme for Textpattern
Offline
Re: r3819 forms nested in fieldset
Considering dropping the fieldsets in favour of div / levers as used everywhere else in the core. Advantage: feels better.
Possible disadvantage: the widgets become collapsible, subject to me putting in a smidgeon of code to retain the settings.
While it’s not a disadvantage per se, having a collapsible control around a single UI element seems a bit of a waste of effort, even if it is consistent with the other panels. But in this case it’s probably better than fugly code hacks or bad validation. Any objections?
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: r3819 forms nested in fieldset
Phil’s pointed me in a better direction than the fieldsets: non-lever <div>s. He’s sending me a new CSS file in the morning so I’ll commit this latest patch tomorrow:
- Fixes fieldset validation in Images/Files panels.
- Fixes a handful of stupid label ID/input control mismatches on the Edit Comment panel (I must have been falling asleep at the keyboard)
- Fixes a couple of label-to-non-input validation warnings on the Edit Author panel.
- Brings the Admin->Import panel to the ‘no tables’ party (at least, on the setup side; the results page is still a list of feedback junk, so the table remains there for the time being). Can anyone test this panel still behaves as it should and report back please?
After that I have some subtle changes to the multi-edit tool in the next couple of days, the new Sections panel is 99.4% done, and I have Gocom’s spiffing shorter, speedier, more cross-browser textpattern.js to roll in. Should have it all finished by the end of the weekend, with luck, then it’s over to everyone else to make sure nothing’s been missed.
Can I just take the opportunity to thank everyone for bearing with us while the cement sets in this wall of changes, and for all the feedback that has helped shape it to date. And a special mention to Phil Wareham who has worked tirelessly auditing the HTML/CSS, mocking up things, and patiently coping with an almost daily barrage of half-arsed patches from me over the past couple of months as we take steps in the general direction of his admin-side utopia. Beer’s on me next time.
Last edited by Bloke (2012-06-22 01:04:00)
The smd plugin menagerie — for when you need one more gribble of power from Textpattern. Bleeding-edge code available on GitHub.
Txp Builders – finely-crafted code, design and Txp
Offline
Re: r3819 forms nested in fieldset
It’s been pretty amazing to watch this unfold. Thanks for your work!
Offline
#10 2012-06-22 07:44:55
- Algaris
- Member
- From: England
- Registered: 2006-01-27
- Posts: 553
Re: r3819 forms nested in fieldset
I completely agree with Marc. I love the open development style that the Textpattern Dev’s and Phil et al. have adopted. Thank you so much for all your hard work.
Offline
Pages: 1