Merge lp://staging/~deadlight/canonical-identity-provider/logout into lp://staging/~deadlight/canonical-identity-provider/vanilla
Proposed by
Karl Williams
| Status: | Merged |
|---|---|
| Merged at revision: | 1691 |
| Proposed branch: | lp://staging/~deadlight/canonical-identity-provider/logout |
| Merge into: | lp://staging/~deadlight/canonical-identity-provider/vanilla |
| Diff against target: |
497 lines (+337/-79) 8 files modified
src/identityprovider/static_src/scss/styles.scss (+8/-0) src/webui/templates/registration/logout.html (+19/-16) src/webui/templates/static/ubuntuone-account.html (+33/-51) src/webui/templates/vanilla/_footer.html (+28/-0) src/webui/templates/vanilla/_header.html (+32/-0) src/webui/templates/vanilla/base.html (+179/-0) src/webui/templates/vanilla/widgets/header-me-menu.html (+18/-0) src/webui/templates/widgets/other_sites.html (+20/-12) |
| To merge this branch: | bzr merge lp://staging/~deadlight/canonical-identity-provider/logout |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Anthony Dillon (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Updated the logout page to use Vanilla styling
---------
QA:
- Visit [HOST]:8000/+logout
NOTE: It's hard to test the message that appears when you are logged into other services but I mocked that up and it looks like this: https:/
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
LGTM +1. Few trivial comments in the code.