Download & Extend

Move themeing of cas login to cas_login.tpl.php

Project:CAS
Version:master
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (won't fix)

Issue Summary

Currently the themeing of the cas login block is hard coded into the module. This patch will move the hard coding to a template file allowing an administrator to theme that block inside their theme folder. It also changes the layout from just a div to a div with an unordered list for the menu item. This is to keep consistency with how other menu items are displayed in drupal.

Attached in the next message will be the patch plus the new cas_login.tpl.php file.

Comments

#1

attachments change cas_login.tpl_.txt to cas_login.tpl.php

AttachmentSizeStatusTest resultOperations
theme login block.patch1.33 KBIgnored: Check issue status.NoneNone
cas_login.tpl_.txt226 bytesIgnored: Check issue status.NoneNone

#2

Status:needs review» needs work

Unfortunately the latest versions of head have completely changed the strategy for this. We're rendering a form with a button instead, so that users can integrate with the persistent login module. The Login menu was added to provide the simple link functionality, so I felt this block was redundant. It would seem to me that these days such things could be provided with a custom menu or block anyway.

Sorry I sat on this for so long.

#3

Status:needs work» closed (won't fix)

At this point I think we've standardized to use the same mechanism as OpenID for the login blocks. Marking as closed, won't fix since this has sat around for so long.