Add direct links to account details and checkout page for logged in user.
Created attachment 33024 [details] [review] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links.
Created attachment 33025 [details] [review] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links.
Created attachment 33026 [details] [review] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links.
Created attachment 33027 [details] [review] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links. Signed-off-by: Magnus Enger <digitalutvikling@gmail.com> Works as advertised. The options are not displayed when you are logged in as the db/admin user.
I can imagine at least some libraries won't want this feature. Can you please add ids and classes to the li tags so they can be easily hidden via css?
Also, is it completely necessary to have links to both moremember.pl and circulation.pl? Eventually we may start running out of real estate on our menu bar ; )
Created attachment 33092 [details] [review] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links. Signed-off-by: Magnus Enger <digitalutvikling@gmail.com> Works as advertised. The options are not displayed when you are logged in as the db/admin user. Added classes "toplinks-myaccount" and "toplink-mycheckouts" to li tags to make it possible to hide them (per Kyle M $ Switching back to "Signd-off" (Hope this is OK becuause it is a tiny string addition) Marc
Sorry, my comment on the patch was mutilated, it should read: ----- Added classes "toplinks-myaccount" and "toplink-mycheckouts" to li tags to make it possible to hide them (per Kyle's request). Switching back to "Signd-off" (Hope this is OK becuause it is a tiny string addition) ----- PS: Showed it today to a possible customer, they like it.
Created attachment 33093 [details] [review] [PASSED QA] Bug 13176 - Add links "My account" and "My checkouts" for logged in user to drop down in staff client header This patch adds links to "My account" and "My checkouts" to drop down in staff client header. To test: Apply patch Got to drop down of logged in user (top right) See new links to "My account" and "My checkout" (above "Log out") Test the links. Signed-off-by: Magnus Enger <digitalutvikling@gmail.com> Works as advertised. The options are not displayed when you are logged in as the db/admin user. Added classes "toplinks-myaccount" and "toplink-mycheckouts" to li tags to make it possible to hide them (per Kyle M $ Switching back to "Signd-off" (Hope this is OK becuause it is a tiny string addition) Marc Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Nice one, thanks Marc!