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

Brave payments: Line should always appear when user scroll down #3888

Closed
bitcoinuser opened this issue Sep 11, 2016 · 8 comments · Fixed by #4395
Closed

Brave payments: Line should always appear when user scroll down #3888

bitcoinuser opened this issue Sep 11, 2016 · 8 comments · Fixed by #4395
Assignees
Labels
design A design change, especially one which needs input from the design team. feature/rewards QA/checked-Linux QA/checked-Win64 suggestion
Milestone

Comments

@bitcoinuser
Copy link

Hi,

I think this line on brave payments should always appear when user scroll down.

captura de tela de 2016-09-11 09-10-53

@charbelrami charbelrami added design A design change, especially one which needs input from the design team. suggestion labels Sep 11, 2016
@mrose17 mrose17 added this to the 0.12.2dev milestone Sep 12, 2016
@mrose17
Copy link
Member

mrose17 commented Sep 12, 2016

@jkup - fyi: this probably should be dealt with at the same time as we deal with #3792

@jkup jkup self-assigned this Sep 13, 2016
@jkup jkup modified the milestones: 0.12.3dev, 0.12.2dev, 0.12.4dev Sep 19, 2016
@jkup
Copy link
Contributor

jkup commented Sep 27, 2016

@bsclifton I think you had some good feedback about this table in chat somewhere but I can't find it. Was it lacking keyboard navigation or something?

@bsclifton
Copy link
Member

bsclifton commented Sep 27, 2016

@jkup Yes! the scrollbar was on the table itself, not on the window. So it wasn't straightforward to scroll through the list (especially without a mouse)

Was able to solve of course by clicking inside that area then scrolling (using gestures or arrow keys)

@jkup
Copy link
Contributor

jkup commented Sep 27, 2016

@bsclifton ah yes! ok great, I'll add that work to this ticket. Thanks.

@bradleyrichter
Copy link
Contributor

bradleyrichter commented Oct 5, 2016

this solution doesn't seem to be working well enough

  • the nested scroll bar now doesn't extend to the bottom of the main window
  • the top of the scroll bar is cropped

image

@jkup Can we instead float the entire top area including the list header row and then use the entire window for scrolling?

@jkup
Copy link
Contributor

jkup commented Oct 5, 2016

@bradleyrichter yeah I mean this is working as expected with the CSS hack I added to make the headers sticky. I agree that it would look much nicer if we didn't set a max-height on the list and just let the window scroll. Should I make a new issue for this? Either way I can get it in this week.

@bradleyrichter
Copy link
Contributor

Sure. Close and make a new issue if you prefer.

On Oct 4, 2016, at 8:42 PM, Jon Kuperman notifications@github.com wrote:

@bradleyrichter yeah I mean this is working as expected with the CSS hack I added to make the headers sticky. I agree that it would look much nicer if we didn't set a max-height on the list and just let the window scroll. Should I make a new issue for this? Either way I can get it in this week.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@bbondy
Copy link
Member

bbondy commented Oct 5, 2016

Reclosing but pls do a new issue since the change would be in 0.12.5 and this is in 0.12.4.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design A design change, especially one which needs input from the design team. feature/rewards QA/checked-Linux QA/checked-Win64 suggestion
Projects
None yet
9 participants