Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Ledger list should not have maximum height #4537

Closed
jkup opened this issue Oct 5, 2016 · 0 comments · Fixed by #4616
Closed

Ledger list should not have maximum height #4537

jkup opened this issue Oct 5, 2016 · 0 comments · Fixed by #4616
Assignees
@jkup
Labels
design A design change, especially one which needs input from the design team. feature/rewards QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Milestone
0.12.5dev

Comments

@jkup
Copy link
Contributor

jkup commented Oct 5, 2016

This relates to #3888

Right now the list has a max height causing it to have a separate scroll container. It would be better if it went the entire page height so there was only a scrollbar on the window.
@jkup jkup self-assigned this Oct 5, 2016
@jkup jkup added the design A design change, especially one which needs input from the design team. label Oct 5, 2016
@jkup jkup added this to the 0.12.5dev milestone Oct 5, 2016
@jkup jkup mentioned this issue Oct 7, 2016
Merged
4 tasks
This was referenced Oct 12, 2016
Closed
Closed
Closed
Closed
Closed
Closed
Closed
This was referenced Oct 14, 2016
Closed
Closed
Closed
Closed
This was referenced Oct 16, 2016
Closed
Closed
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jkup jkup

Labels
design A design change, especially one which needs input from the design team. feature/rewards QA/checked-macOS QA/checked-Win32 QA/checked-Win64
Projects
None yet
Milestone
0.12.5dev
Development

Successfully merging a pull request may close this issue.

Ledger list should not have maximum height brave/browser-laptop
3 participants
@jkup @alexwykoff @luixxiul