Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
theme system
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
17 Aug 2009 at 05:37 UTC
Updated:
1 Nov 2009 at 21:20 UTC
The current way IA is going, it creates major regression when toolbar.module is disabled (default on expert profile)
Setup:
- Set Seven as admin theme.
- Disable Management Menu Block in Seven.
- Ensure toolbar.module is disabled.
Rules:
- Don't type the url in the location bar of your browser.
Challenges:
- Go to the user listing
- Go to the content listing
- Enable an module
- Change the theme
if you try to do the same on D6, you find that each item is doable while on D7 it is (nearly) impossible to do!
This issue is the central hub in fixing those issues.
Comments
Comment #1
alexanderpas commentedSolutions:
- breadcrumbs allow you to go to a less drilled down view.
- local tasks shoudn't be part of categories (except for view change).
- Everything should be contained in a Category
- Category listings should basically only use
'page callback' => 'system_admin_menu_block_page',and'access callback' => 'system_admin_menu_block_access',besides the obvious onesIssues:
#548806: No breadcrumbs in Seven
#550718: admin/people needs to have a default tab (was: Tabs on admin pages are not accessible from overviews and menus)
#551080: List non-container items on /admin for a complete overview
Notes:
I've talked with webchick about this in IRC, and this should preferably solved before freeze
IA is required to be working without toolbar.module
Comment #2
Bojhan commentedLets get to the point. Its work in progress so obviously it won't be working very elegant yet. But I don't see how we couldn't fix these issues.
Comment #3
gábor hojtsyThese do not stand. Drupal 6 had a rule to have everything in a container, but this is not true anymore in D7. Of course (or unfortunately), the /admin listing code was not updated to list items which are not in a category, but that should be fixed indeed. Provided the same feedback on #551080: List non-container items on /admin for a complete overview which can be generalized for this. That one applies to People, Content and other items just like Appearance.
Comment #4
alexanderpas commentedIn the current situation, these do stand, HEAD in it's current state is unworkable (and undeliverable) from a usability standpoint, and we're only two weeks away from freeze.
the most important questions to ask (yourself) now are:
- where is (was) the issue to fix /admin in the first place
- why are we, so close to freeze implementing radical changes, when a keystone hasn't been created yet
- why can't the dashboard simply live at /admin/dashboard, and be the default local task (just like /admin/by-task currently is.), in addition to the two we already have (/admin/by-module & /admin/by-task)
the only issue i could find about this is: #376103: Make /admin into a REAL dashboard.
toolbar.module takes about 20% of the real estate that is left for the page on my eeePC
Comment #5
gábor hojtsy@alexanderpas: I more then hear you :) Just repurposed #551080: List non-container items on /admin for a complete overview to fix the /admin page. It does not solve #550718: admin/people needs to have a default tab (was: Tabs on admin pages are not accessible from overviews and menus), but that is why that one is another issue. I agree some of the IA changes were implemented without aligning our original overview screens, but its not like there is no time to fix that. It is a bug after all, so it would be fixed after code freeze anyway. (Not that we should not fix it as soon as possible, so I did provide a patch on #551080: List non-container items on /admin for a complete overview).
Comment #6
xmacinfo@alexanderpas I agree with you. Too many people makes the assumption that the toolbar will always be enabled or that Seven will be the admin theme. I'm glad you take time to raise issues before things gets unmanageable.
Comment #7
alexanderpas commented