It could use some ID's beyond the yui classes.
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!