I think that the current UI for the expert field template is very confusing because you have lots of fields that you probably don't need because the corresponding checkbox isn't checked.

That was the reason for writing a small patch which adds the #states to all wrapper fields.

Before the patch was applied:
ds-field-template-expert-before.pngAfter the patch was applied:
ds-field-template-expert-patched.png

And here you can see how it looks like when checking a checkbox:
ds-field-template-expert-patched-1.png

Comments

Status:Active» Needs review
StatusFileSize
new3.41 KB

Status:Needs review» Reviewed & tested by the community

This looks like a good usability improvement to me. Code is fine and working like expected. Nice usage of the already existing checkbox :)

Status:Reviewed & tested by the community» Needs work
StatusFileSize
new61.87 KB

I'm going to be annoying here - but I do like the idea, it's been bothering me a lot as well.
I like the idea, but let's try to fix the following then too:

  • Figure out if we can do the same with label (maybe hide by default and show once there's a label in it)
  • Fix the alignment on DS fields, e.g. title, see screenshot - I *think* it has something todo with colspans

I'll have a look myself after next week unless someone beats me to it.

StatusFileSize
new140.3 KB

I think we could add a new checkbox called "Label" what do you think? What do you mean by colspans? I cannot see any colspans...
Screen Shot 2013-03-05 at 00.04.54.png

Great idea. I'm seeing the same problem as @swentel in terms of the wrapping, though it's completely unrelated to this particular issue.

I went ahead and created a separate issue for that #2009496: Expert field template has broken styles in the admin

(Though, if it can be fixed here, why not?)

Category:feature» task

Status:Needs work» Needs review
StatusFileSize
new4.02 KB
new98.13 KB

We should get this going again...

I'm not sure how to solve swentel's issue with the label fields, but I've added some styling to the rest that I think makes things much better. I've also added overflow:auto; to the .ft-group css so that the clearfix's are no longer necessary. I've also fixed the funky wrapping with a whitespace:normal; to override the nowrap from the system css.

I've rolled those changes with the patch from above.

Cool! Tested your patch and overflow: auto; works but I don't know whether the border is a good solution :/ Let's see what swentel thinks about that :)

can your provide a patch and screenshot without the border?

Strange, just tested my patch form #1 and it works fine.

I guess that adding an overflow: auto; and white-space: normal; is not necessary :/

StatusFileSize
new185.86 KB
new179.15 KB

No bordersBorders

Same shot, with and without borders. I still think they help clarify the sections.

yannickoo, have you tried resizing your browser window? It might look ok when you're wide enough to see everything in one line, but as soon as the fields start wrapping things look funky without those styles. Also, overflow:auto is generally good practice when you have a div containing floats.

This on top of my list together with the label wrapper. Will fix this when I have some time.

Issue tags:+sprint-nascom-2013

tagging for sprint

Version:7.x-2.x-dev» 8.x-2.x-dev
Issue summary:View changes
Status:Needs review» Patch (to be ported)

Fixed this, needs porting