Modify

Opened 4 years ago

Closed 6 months ago

#10739 closed enhancement (fixed)

[Patch] Move Back to Accounts button to the contextual navigation

Reported by: Ryan J Ollos Owned by: Steffen Hoffmann
Priority: normal Component: AccountManagerPlugin
Severity: minor Keywords: web-UI context-navigation
Cc: Trac Release: 0.11

Description (last modified by Ryan J Ollos)

The attached patch moves the Back to Accounts button on the Accounts: Cleanup and Review User Account Details administration pages to the contextual navigation.

Attachments (1)

th10739-r12491-1.patch (3.4 KB) - added by Ryan J Ollos 4 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 4 years ago by Ryan J Ollos

Description: modified (diff)

I'll post the patch shortly.

Changed 4 years ago by Ryan J Ollos

Attachment: th10739-r12491-1.patch added

comment:2 Changed 4 years ago by Ryan J Ollos

Description: modified (diff)

The attached patch applies to two account administration pages.

comment:3 Changed 4 years ago by Steffen Hoffmann

Keywords: web-UI context-navigation added
Severity: normalminor
Trac Release: 0.11

Nice enhancement, like it. Thanks for sharing your web-UI design experience.

comment:4 Changed 4 years ago by Steffen Hoffmann

(In [12492]) AccountManagerPlugin: Move back-link to user admin main page to Trac context navigation, refs #10739.

This is a nice web-UI simplification. Thanks to Ryan J Ollos for suggestion and patch code.

comment:5 Changed 4 years ago by Ryan J Ollos

Maybe the text should be Back to Manage User Accounts? More descriptive and accurate, sure, but probably too long for a contextual navigation item. Just thought I'd throw the idea out.

comment:6 in reply to:  5 Changed 4 years ago by Steffen Hoffmann

Replying to rjollos:

Maybe the text should be Back to Manage User Accounts? More descriptive and accurate, sure, but probably too long for a contextual navigation item.

Sure, I've thought about that for quite some time, but the shortness really matters a bit more than total accuracy for what I expect.

Just thought I'd throw the idea out.

Oh, perfectly valid reasoning. But I hope we can still agree to leave that as it is.

comment:7 Changed 4 years ago by Ryan J Ollos

Yes, agreed. Just wanted to make sure you didn't have a better idea for improving this. I can't think of a better way at the moment.

comment:8 Changed 6 months ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.