I noticed that tha table themed admin interface for table style output was very difficult to follow, too wide and too not fit for the views admin interface, so attached is a, IMHO, more inline with views admin interface.
First of all, I fixed "collapsable" for "collapsible" as that's the correct FAPI attribute.
Then I put all the field table options in a fieldset and have that not collapsed. Then have all of the other settings per fields collapsed except for the position settings.
I also did something similar to the unformatted field settings, although this one was actually more into this line of things, so I mostly give both a similar look and feel for the configuration settings.
I also removed the theme function for styling this and hence, the viuews_pdf.admin.inc.
Will submit patch now.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1349442-1-better-admin-interface.patch | 11.69 KB | hanoii |
Comments
Comment #1
hanoiiComment #2
Simon Georges CreditAttribution: Simon Georges commentedAs I committed patch from #1270346-1: collapsable should be collapsible (Typing error) (smaller patch, easier to review / commit), this needs a re-roll. I'll handle it if nobody does before ;-)
Comment #3
Simon Georges CreditAttribution: Simon Georges commentedComment #4
killua99 CreditAttribution: killua99 commentedExist a patch to handle this issues and others. Please see #2032189: Multiple issue fixing in a single patch and review others issues patch.
This is RTBC, and ready to apply.
Comment #5
killua99 CreditAttribution: killua99 commentedComment #6
killua99 CreditAttribution: killua99 commentedTagging for the next version.
Comment #7
killua99 CreditAttribution: killua99 commentedComment #8
killua99 CreditAttribution: killua99 commentedComment #9
vegansupreme CreditAttribution: vegansupreme commentedI think this was fixed a while ago with #2032189: Multiple issue fixing in a single patch
All of the changes mentioned in the OP have been included into the module for a long time now.
If we want to update the UI for V2, what specifically should change? Should this be a new issue?
Comment #10
killua99 CreditAttribution: killua99 commentedEh yes, this will need a new issue. But will be under "i+d" label with low priority tho