Modify

Opened 11 years ago

Closed 7 years 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 11 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by Ryan J Ollos

Description: modified (diff)

I'll post the patch shortly.

Changed 11 years ago by Ryan J Ollos

Attachment: th10739-r12491-1.patch added

comment:2 Changed 11 years ago by Ryan J Ollos

Description: modified (diff)

The attached patch applies to two account administration pages.

comment:3 Changed 11 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 11 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 11 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 11 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 11 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 7 years 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.