Skip to content

fix "create folder" icon overlaying home icon - #17017

Merged
rullzer merged 1 commit into
nextcloud:masterfrom
ritsute:fix/css-actions.creatable
Sep 19, 2019
Merged

fix "create folder" icon overlaying home icon#17017
rullzer merged 1 commit into
nextcloud:masterfrom
ritsute:fix/css-actions.creatable

Conversation

@ritsute

@ritsuteritsute commented Sep 5, 2019

Copy link
Copy Markdown
Contributor

First contribution. Home icon is blocked. I've edited CSS from absolute to relative and adjusted position accordingly.

Steps to reproduce
Go to any uploaded files and right click on it, select "Move or Copy"

Actual behaviour
"Create new folder" icon is overlaying "home" icon.
image

Expected behaviour
"Create new folder" icon should be to the left of "span.dirtree.breadcrumb", not covering "Home" icon.
image

Server configuration
Operating system:
Ubuntu 18.04

Web server:
apache2 2.4.29

Database:
mysql 5.8

PHP version:
7.2.19

Nextcloud version: (see Nextcloud admin page)
16.0.4

Updated from an older Nextcloud/ownCloud or fresh install:
fresh installed. Currently updates using settings/admin/overview button.

Where did you install Nextcloud from:
single php file from nextcloud deploy.

@kesselbkesselb added 3. to review Waiting for reviews bug labels Sep 5, 2019
@kesselbkesselb added this to the Nextcloud 18 milestone Sep 5, 2019
@kesselb

Copy link
Copy Markdown
Contributor

Thank you @ritsute 👍

Would you mind to sign off your commits? https://github.com/nextcloud/server/pull/17017/checks?check_run_id=212969543 for more details.

Signed-off-by: Brandon <brandon.yeow@websparks.sg>
@ritsute
ritsuteforce-pushed the fix/css-actions.creatable branch from 2a6c8b6 to a67941fCompareSeptember 6, 2019 05:50

@juliusknorrjuliusknorr 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.

Tested and works

@juliusknorrjuliusknorr added the 4. to release Ready to be released and/or waiting for tests to finish label Sep 17, 2019
@juliusknorr

Copy link
Copy Markdown
Member

/backport to stable17

@juliusknorr

Copy link
Copy Markdown
Member

/backport to stable16

@skjnldsvskjnldsv removed the 3. to review Waiting for reviews label Sep 17, 2019
@rullzer
rullzer merged commit 505fc0e into nextcloud:masterSep 19, 2019
@welcome

welcomeBot commented Sep 19, 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!

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable17 in #17219

@kesselb

Copy link
Copy Markdown
Contributor

/backport to stable16

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #17227

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@ritsute@kesselb@juliusknorr@skjnldsv@rullzer