Skip to content

[stable13] Fix missing clipboard icon in shared links - #8034

Merged
rullzer merged 1 commit into
stable13from
13-8032
Jan 24, 2018
Merged

[stable13] Fix missing clipboard icon in shared links#8034
rullzer merged 1 commit into
stable13from
13-8032

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

The clipboard icon in shared links appears either directly on the link
input field or, if any social sharing app is enabled, in a menu. The
clipboard icon uses the same CSS rules as other icons (like the
information icon) to be posioned on the end of the input field, and
those rules have to be "cancelled" when the icon is shown in the menu.

Fixes#7990

Backport of #8032

The clipboard icon in shared links appears either directly on the link
input field or, if any social sharing app is enabled, in a menu. The
clipboard icon uses the same CSS rules as other icons (like the
information icon) to be posioned on the end of the input field, and
those rules have to be "cancelled" when the icon is shown in the menu.
Fixes#7990
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Jan 24, 2018
@MorrisJobkeMorrisJobke added this to the Nextcloud 13 milestone Jan 24, 2018
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

I tested it and it works 👍

@MorrisJobkeMorrisJobke mentioned this pull request Jan 24, 2018
5 tasks

@danxuliudanxuliu 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 👍

@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 24, 2018
@codecov

codecovBot commented Jan 24, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8034 into stable13 will increase coverage by 0.16%.
The diff coverage is n/a.

@@ Coverage Diff @@## stable13 #8034 +/- ##
==============================================
+ Coverage 51.23% 51.39% +0.16% + Complexity 24981 24978 -3 
==============================================
Files 1607 1599 -8 Lines 95049 94718 -331 Branches 1376 1376 ==============================================
- Hits 48695 48685 -10 + Misses 46354 46033 -321
Impacted FilesCoverage ΔComplexity Δ
apps/user_ldap/lib/Notification/Notifier.php
apps/testing/composer/composer/ClassLoader.php
core/js/sharedialogexpirationview.js
...trashbin/composer/composer/autoload_namespaces.php
core/Command/Config/App/Base.php
lib/public/Defaults.php
lib/private/Share/SearchResultSorter.php
apps/oauth2/templates/admin.php
apps/updatenotification/composer/autoload.php
apps/federation/composer/autoload.php
... and 3196 more

@rullzer
rullzer merged commit d76f08d into stable13Jan 24, 2018
@rullzer
rullzer deleted the 13-8032 branch January 24, 2018 18:19
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 finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@MorrisJobke@danxuliu@rullzer