Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Jan 24, 2018
Merged

Conversation

MorrisJobke
Copy link
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>
@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Jan 24, 2018
@MorrisJobke MorrisJobke added this to the Nextcloud 13 milestone Jan 24, 2018
@MorrisJobke
Copy link
Member Author

I tested it and it works 👍

@MorrisJobke MorrisJobke mentioned this pull request Jan 24, 2018
5 tasks
Copy link
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

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

Tested and works 👍

@MorrisJobke MorrisJobke 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
Copy link

codecov bot commented Jan 24, 2018

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 Files Coverage Δ 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 stable13 Jan 24, 2018
@rullzer rullzer deleted the 13-8032 branch January 24, 2018 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready 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