Make WordPress Core

Opened 11 years ago

Last modified 6 years ago

#26311 new enhancement

Updated (responsive) Tools -> Export admin screen

Reported by: netweb's profile netweb Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.8
Component: Export Keywords: needs-refresh
Focuses: ui Cc:

Description

Updated Tools -> Export admin screen

Switched from un-ordered lists based html to tables based HTML based on Permalinks admin screen.

No javascript was harmed in this patch...

Attachments (5)

26311.diff (10.1 KB) - added by netweb 11 years ago.
Tools-Export-New.png (38.6 KB) - added by netweb 11 years ago.
New screen
Tools-Export-Original.png (32.5 KB) - added by netweb 11 years ago.
Original screen
Tools-Export-New-CPT.png (39.0 KB) - added by netweb 11 years ago.
New with CPT
Tools-Export-300px.png (17.9 KB) - added by netweb 11 years ago.
New @ 300px width

Download all attachments as: .zip

Change History (13)

@netweb
11 years ago

@netweb
11 years ago

New screen

@netweb
11 years ago

Original screen

@netweb
11 years ago

New with CPT

@netweb
11 years ago

New @ 300px width

#1 @SergeyBiryukov
11 years ago

  • Keywords ui-focus added

#2 @nacin
11 years ago

Switching *to* a table? Let's try to solve this another way.

#3 @netweb
11 years ago

I see dead people tables, If you can't beat 'em, join 'em

I see where your coming from regarding tables and I wasn't going to go this route, then I took a look around...

I had hoped to find some of these already MP6'ised aside from Menus, Themes & Widgets but nope, not much else has been updated to use styles based on MP6.

Admin Dashboard Section HTML Style
Dashboard -> HomeCustom
Dashboard -> UpdatesList
Posts -> All Postsdiv heading with tables
Posts -> Add Newdiv head with embed tinymce
Posts -> Categoriesdivs
Posts -> Tagsdivs
Media -> Librarydiv heading with tables
Media -> Add Newdivs
Pages -> All Postsdiv heading with tables
Pages -> Add Newdiv head with embed tinymce
Commentsdiv heading with tables
Appearance -> Themescustom divs
Appearance -> Customizecustom
Appearance -> Widgetsdivs
Appearance -> Menusdivs and lists
Appearance -> Headerdiv heading with tables
Appearance -> Backgrounddiv heading with tables
Appearance -> Editordiv with textarea form
Plugins -> Installed Pluginsdiv heading with tables
Plugins -> Add Newdiv with form and paragraph tag cloud
Plugins -> Editordiv with textarea form
Users -> All Usersdiv heading with tables
Users -> Add Newdiv heading with tables
Users -> Your Profilediv heading with tables
Tools -> Available Toolsdiv with paragraph
Tools -> Importdiv heading with tables
Tools -> ExportSee patch attached to this ticket
Settings -> Generaldiv heading with tables
Settings -> Writingdiv heading with tables
Settings -> Readingdiv heading with tables
Settings -> Discussiondiv heading with tables
Settings -> Mediadiv heading with tables
Settings -> Permalinksdiv heading with tables

More than happy if you want to close the ticket as invalid/wontfix as I haven't really got any more thoughts (more specifically code ideas) without completely removing tables from all of the sections in the list.

This really then leads to waiting for a 'Stlye Guide' to be in place first. Helen has started this wp-style-guide (Tables, everywhere I tell you) but is in need of some updates and probably a discussion had on the Make Core/UI blog to facilitate some ideas.

#5 @chriscct7
9 years ago

  • Keywords needs-refresh added

@ryanmccue is this going to be covered as part of your import/export improvments?

#7 @rmccue
9 years ago

@chriscct7 I haven't done any UI work yet, and in fact, the UI is completely untested. I'm also only working on the importer at the moment.

#8 @hlashbrooke
9 years ago

Just a note to say that the patch proposed in #27048 also fixes the issue discussed in this ticket as the updated layout of the page looks just fine on mobile.

Note: See TracTickets for help on using tickets.