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.

CommentFileSizeAuthor
#15 movestats-1.patch7.83 KBjoshmiller
#9 system-category.patch24.52 KBgábor hojtsy
#6 System.png23.67 KBgábor hojtsy
#5 system-category.patch24.18 KBgábor hojtsy

Comments

yoroy’s picture

yoroy commented
Issue tags: +IA

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

yoroy’s picture

yoroy commented

Part of #546956: [meta-issue] Overhaul of Information Architecture

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Actions, actions! Actions, too!

catch’s picture

catch
he/him
Primary language English
commented

+1

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Needs review
StatusFileSize
new system-category.patch24.18 KB

Ok, so moving site information and actions.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
StatusFileSize
new System.png23.67 KB

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

Status: Needs review » Needs work

The last submitted patch failed testing.

yoroy’s picture

yoroy commented
Status: Needs work » Needs review

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

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
StatusFileSize
new system-category.patch24.52 KB

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.

catch’s picture

catch
he/him
Primary language English
commented

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

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Forgot status change.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
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.

Bojhan’s picture

Bojhan commented

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.

yoroy’s picture

yoroy commented

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

joshmiller’s picture

joshmiller
he/him
Primary language English
Location Indianapolis, Indiana, USA
commented
Status: Needs work » Needs review
StatusFileSize
new movestats-1.patch7.83 KB

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!

douggreen’s picture

douggreen commented
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.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Ok, committed!!

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

Bojhan’s picture

Bojhan commented
Status: Fixed » Needs work

*cough*

traviscarden’s picture

traviscarden commented

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

yoroy’s picture

yoroy commented
Status: Needs work » Fixed

voila

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -Needs documentation, -IA, -D7UX

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Usability, +Needs documentation, +IA, +D7UX

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.