Summary: | logged in and logged out states of opac-main.tt are hard to style | ||
---|---|---|---|
Product: | Koha | Reporter: | Liz Rea <wizzyrea> |
Component: | OPAC | Assignee: | Liz Rea <wizzyrea> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, koha.sekjal, paul.poulain, wizzyrea |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 7128 - adding some ID's to make styling opac-main.tt a bit easier
Bug 7128 - adding some ID's to make styling opac-main.tt a bit easier Bug 7128 : Follow up, adding the id to the case when a user is not logged in Signed-off patches, squashed |
Description
Liz Rea
2011-11-01 19:09:16 UTC
Created attachment 6133 [details] [review] Bug 7128 - adding some ID's to make styling opac-main.tt a bit easier To test: verify that the loggedin id is added when user is logged in verify that the notloggedin id is evident when no user is logged in. Created attachment 6156 [details] [review] Bug 7128 - adding some ID's to make styling opac-main.tt a bit easier To test: verify that the loggedin id is added when user is logged in verify that the notloggedin id is evident when no user is logged in. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Created attachment 6157 [details] [review] Bug 7128 : Follow up, adding the id to the case when a user is not logged in Liz your patch actually only sets the id for not logged in, when we don't allow login at all. Rather than if people are just not logged in, I have attached a follow up patch, to add the id to the other case (and fixed the formatting to make it clearer) can you take a look at that please? Created attachment 6211 [details] [review] Signed-off patches, squashed Template only change, easy to read, marking as Passed QA. Huzzah for more IDs! patch pushed, please test ah, I see that now, thanks all! |