Configuration page: System category

Bojhan - August 31, 2009 - 17:04
Project:Drupal
Version:7.x-dev
Component:base system
Category:task
Priority:normal
Assigned:Unassigned
Status:needs work
Issue tags:D7UX, IA, Needs Documentation, Usability
Description

We need to create a System category to house Site Information. Because this is merely site related, they should be grouped together. Its inevitable that we need a category like this, and creating it to house these items gives some sense that module maintainers shouldn't put their modules here.

#1

yoroy - August 31, 2009 - 17:20
Issue tags:+IA

Which items in core would go here?
Site information, and actions?, …?

#2

yoroy - August 31, 2009 - 17:29

#3

Gábor Hojtsy - August 31, 2009 - 17:33

Actions, actions! Actions, too!

#4

catch - August 31, 2009 - 17:42

+1

#5

Gábor Hojtsy - August 31, 2009 - 18:50
Status:active» needs review

Ok, so moving site information and actions.

AttachmentSize
system-category.patch 24.18 KB
Testbed results
system-category.patchfailedFailed: 12728 passes, 2 fails, 7 exceptions Detailed results

#6

Gábor Hojtsy - August 31, 2009 - 18:54

And the obligatory screenshot to get "the feel of it" :)

AttachmentSize
System.png 23.67 KB

#7

System Message - August 31, 2009 - 20:17
Status:needs review» needs work

The last submitted patch failed testing.

#8

yoroy - August 31, 2009 - 21:58
Status:needs work» needs review

Can we weight this a bit already and make site info show up on top?

#9

Gábor Hojtsy - September 1, 2009 - 06:50

Of course I forgot to update a menu item page callback index which resulted in test fails. Fixed in attached patch.

@yoroy: fixed that too.

AttachmentSize
system-category.patch 24.52 KB
Testbed results
system-category.patchfailedFailed: Failed to apply patch. Detailed results

#10

catch - September 1, 2009 - 07:15

Works for me, I expect actions will move somewhere else eventually, but let's get it out of admin/settings to start with.

#11

catch - September 1, 2009 - 07:15
Status:needs review» reviewed & tested by the community

Forgot status change.

#12

webchick - September 1, 2009 - 16:58
Status:reviewed & tested by the community» needs work
Issue tags:+Needs Documentation

Committed to HEAD. Woot!

I actually like Actions here, and would suggest moving Trigger. Both are general-purpose tools that can be used in a variety of contexts.

Needs work for docs.

#13

Bojhan - September 3, 2009 - 15:04

I think we missed Statistics, it seams reasonable to move that - almost all its settings are related to the system and do not fit anywhere else very well. I think we could discuss endlessly how to improve Statistics and perhaps move it somewhere else - but its hard to reach consensus on that.

I spoke with catch on this as well, and he was al right with it - since it eliminates our last section, we can always come back to it.

#14

yoroy - September 5, 2009 - 10:19

yep, go ahead and move statistics to system category please.

#15

joshmiller - September 5, 2009 - 12:01
Status:needs work» needs review

yoroy / bojhan -

Attached is a patch that moves statistics to the right URL...

To test:

1) install Drupal HEAD
2) Turn on Statistics module
3) navigate to 'admin/config'
4) click on "Statistics"

Works for me!

AttachmentSize
movestats-1.patch 7.83 KB
Testbed results
movestats-1.patchpassedPassed: 12789 passes, 0 fails, 0 exceptions Detailed results

#16

douggreen - September 5, 2009 - 13:31
Status:needs review» reviewed & tested by the community

I reviewed this patch, and it looks fine. The move from admin/settings/statistics to admin/config/system/statistics is required in Drupal 7. I confirmed that all the links appear on the right menu and work.

#17

webchick - September 5, 2009 - 13:50
Status:reviewed & tested by the community» fixed

Ok, committed!!

Now let's quick get a patch to remove Site configuration! Hooray!! :D

#18

Bojhan - September 5, 2009 - 14:35
Status:fixed» needs work

*cough*

#19

TravisCarden - September 9, 2009 - 19:15

@webchick: They're working on such a patch at #569282: Configuration page: Removal of Site configuration

 
 

Drupal is a registered trademark of Dries Buytaert.