User tests: Successful: Unsuccessful:
This PR moves the Filter option of the Redirect Manager: Links to the [Search Tools] button.
(See Issue #6941 regarding the inconsistency with Filter options).
In back-end > Components > Redirects
See the filter "- Select Status-" on the left
Also notice that the Expired URL column is too small.
Compare with the other filters, e.g. in Category Manager:
In back-end > Content > Categories
from the left to the middle column and will be visible when you click on the [Search Tools] button.
This PR also fixes the column width of the Expired URL column (now the columns Expired URL, New URL, and Referring Page are all 30%) so that it looks better on a smaller screen.
Labels |
Added:
?
|
Labels |
Added:
?
|
Category | ⇒ | Administration UI/UX |
Status | New | ⇒ | Pending |
Rel_Number | 0 | ⇒ | 6941 |
Relation Type | ⇒ | Related to |
@pe7er i have just tested this.
I hope we are able to remove the side bar as it looks very bad.
The seccond thing i marked is the info box are we abel to add a bit space between the searchtools and the message? If searchtools are open all is ok but can we have this extra space also if the searchtools are closed?
Also i have just noted that the filter don't work. It will be shown but don't work here. If i change to e.g. unpublished i still get all also the publiched etc.
Labels |
Added:
?
|
Thank you all for testing.
I've committed the PR from @zero-24 to fix a CS issue,
and after changing the field name in the filter.xml (thanks @bertmert !) the filter works correctly.
Do I still need to add "config_fields" to the constructor of the model?
I also added a "Links" submenu so that (although this component has just 1 view) the view is consistent with all other components.
Menu on left contains "Links" now. "Search Tools" works when filtering. Spacing between "search tools" and the below message still is not corrected.
I do not understand why there is a sidebar with a link to open the view that you are already on - this serves no purpose at all
Consistency & it's a visual aid that makes orientation (especially for new users) easier.
All components that have multiple views (e.g. items & categories) have a sidebar with (sort of) "menu items" where you can easily select another view of that component. Besides selecting "menu options" it also shows the admin which "admin sub menu" they are using.
I've noticed that especially for new users the admin menu structure can be very confusing. The sidebar functions as visual aid that makes orientation easier.
And when you only have one highlighted (blue background) option, it gives admins a visual aid that is consistent with all "multi view" components.
I understand you but disagree.
On 23 May 2015 1:35 pm, "Peter Martin" notifications@github.com wrote:
Consistency & it's a visual aid that makes orientation (especially for new
users) easier.All components that have multiple views (e.g. items & categories) have a
sidebar with (sort of) "menu items" where you can easily select another
view of that component. Besides selecting "menu options" it also shows the
admin which "admin sub menu" they are using.I've noticed that especially for new users the admin menu structure can be
very confusing. The sidebar functions as visual aid that makes orientation
easier.And when you only have one highlighted (blue background) option, it gives
admins a visual aid that is consistent with all "multi view" components.—
Reply to this email directly or view it on GitHub
#6953 (comment).
I would remove the sidebar, too, because any unneccessary rendering ist a waste of time and performance. Every little helps...
Tested succesfully. I agree however, that the sidebar is only useful when an item has subitems. With Redirect, that's not the case. And why use the term "Links" and not "Redirect"?
@test succesfully tested. No sidebar necessary IMHO.
The space between searchtools and message bar should be fixed?
Milestone |
Added: |
Status | Pending | ⇒ | Information Required |
Moving to Information Required
until @pe7er get the time to change the Code as @roland-d sugsested.
Closing as we have a new & updated.
Status | Information Required | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-17 07:08:54 |
Closed_By | ⇒ | zero-24 |
Milestone |
Removed: |
@test
able to reproduce then #6953 works as described (Travis) - thanks
strange having the left collapse column empty O.o