Merge lp:~maxiberta/canonical-identity-provider/applications-prevent-scrollbar into lp:canonical-identity-provider/release

Proposed by Maximiliano Bertacchini
Status: Merged
Approved by: Maximiliano Bertacchini
Approved revision: no longer in the source branch.
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~maxiberta/canonical-identity-provider/applications-prevent-scrollbar
Merge into: lp:canonical-identity-provider/release
Diff against target: 12 lines (+1/-1)
1 file modified
src/webui/templates/account/applications.html (+1/-1)
To merge this branch: bzr merge lp:~maxiberta/canonical-identity-provider/applications-prevent-scrollbar
Reviewer Review Type Date Requested Status
Maximiliano Bertacchini Approve
Karl Williams (community) Approve
Review via email: mp+379211@code.launchpad.net

Commit message

Prevent scrollbars around the apps list when not needed.

To post a comment you must log in.
Revision history for this message
Karl Williams (deadlight) wrote :

Looks good. +1

review: Approve
Revision history for this message
Otto Co-Pilot (otto-copilot) wrote :
Revision history for this message
Maximiliano Bertacchini (maxiberta) wrote :

We trust Karl's approval.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/webui/templates/account/applications.html'
2--- src/webui/templates/account/applications.html 2020-01-22 12:52:26 +0000
3+++ src/webui/templates/account/applications.html 2020-02-14 13:59:24 +0000
4@@ -22,7 +22,7 @@
5 {% blocktrans %}You can choose to allow an application to have access again if you like by opening that application and instructing it to access your account. You'll have to provide your username and password before it will be granted access.{% endblocktrans %}
6 </p>
7 <h3>{% trans "Applications you last used" %}</h3>
8- <div style="overflow: scroll">
9+ <div>
10 <table class="p-table" style="min-width: 28em">
11 <thead>
12 <tr>