Skip to content

Position fixed for .toastify.toast - #17110

Merged
rullzer merged 1 commit into
nextcloud:masterfrom
zorn-v:patch-1
Sep 15, 2019
Merged

Position fixed for .toastify.toast#17110
rullzer merged 1 commit into
nextcloud:masterfrom
zorn-v:patch-1

Conversation

@zorn-v

Copy link
Copy Markdown
Member

If you have page with scrolling and some button on bottom of it, which show notification, you just do not see that notification now.

@rullzer

Copy link
Copy Markdown
Member

Do you have screenshots to show it?

@rullzerrullzer added 3. to review Waiting for reviews design Design, UI, UX, etc. enhancement labels Sep 14, 2019
@rullzerrullzer added this to the Nextcloud 18 milestone Sep 14, 2019

@skjnldsvskjnldsv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I noticed it as well :D

@rullzer
rullzer merged commit d231fc9 into nextcloud:masterSep 15, 2019
@welcome

welcomeBot commented Sep 15, 2019

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
Most developers hang out on IRC. So join #nextcloud-dev on Freenode for a chat!

@zorn-v
zorn-v deleted the patch-1 branch September 16, 2019 01:07
@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable17

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable17 in #17300

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

Labels

3. to reviewWaiting for reviewsdesignDesign, UI, UX, etc.enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@zorn-v@rullzer@skjnldsv