Skip to content

dashboard should not have a scrollbar by default - #33954

Merged
szaimen merged 1 commit into
masterfrom
enh/33868/dashboard-scrollbar
Sep 9, 2022
Merged

dashboard should not have a scrollbar by default#33954
szaimen merged 1 commit into
masterfrom
enh/33868/dashboard-scrollbar

Conversation

@szaimen

@szaimenszaimen commented Sep 8, 2022

Copy link
Copy Markdown
Contributor

Close#33868

BeforeAfter
imageimage

Signed-off-by: szaimen szaimen@e.mail.de

@szaimenszaimen added the 2. developing Work in progress label Sep 8, 2022
@szaimenszaimen added this to the Nextcloud 25 milestone Sep 8, 2022
@szaimen

Copy link
Copy Markdown
ContributorAuthor

/compile amend /

Signed-off-by: szaimen <szaimen@e.mail.de>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command
nextcloud-commandforce-pushed the enh/33868/dashboard-scrollbar branch from a2aa7e9 to 079d444CompareSeptember 8, 2022 07:54
@szaimenszaimen added ui-refresh-feedback 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 8, 2022
@szaimen
szaimen marked this pull request as ready for review September 8, 2022 07:59
@szaimen
szaimen requested review from a team, PVince81, Pytal and skjnldsv and removed request for a teamSeptember 8, 2022 07:59
@blizzzblizzz mentioned this pull request Sep 8, 2022
@szaimen

Copy link
Copy Markdown
ContributorAuthor

CI failure unrelated

@szaimen
szaimen merged commit aa53ded into masterSep 9, 2022
@szaimen
szaimen deleted the enh/33868/dashboard-scrollbar branch September 9, 2022 06:50
@blizzzblizzz mentioned this pull request Sep 9, 2022
@AndyXheli

AndyXheli commented Sep 9, 2022

Copy link
Copy Markdown
Contributor

Hey guy this is still present on beta 6. Should i open a new ticket ? or is it being implemented in #33993
image

@szaimen

Copy link
Copy Markdown
ContributorAuthor

or is it being implemented in #33993

Yes

@AndyXheli

Copy link
Copy Markdown
Contributor

Thanks @szaimen sorry to bother.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dashboard app has an outer scrolling but shouldn't have one

4 participants

@szaimen@AndyXheli@skjnldsv@Pytal