Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
Fix copy button icon overlapping with copyable text (#10227)
Browse files Browse the repository at this point in the history
* Modified css for copy button and the copyable text accross the app specially to solve the overlapping problem in the access token div

* Modified css for copy button and the copyable text accross the app specially to solve the overlapping problem in the access token div and replaced the scroll with auto
  • Loading branch information
Adesh-Pandey authored Feb 23, 2023
1 parent 8f7f855 commit 9a0e537
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
6 changes: 4 additions & 2 deletions res/css/views/elements/_CopyableText.pcss
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,12 @@ limitations under the License.
max-width: 100%;

&.mx_CopyableText_border {
overflow: auto;
border-radius: 5px;
border: solid 1px $light-fg-color;
margin-bottom: 10px;
margin-top: 10px;
padding: 10px;
padding: 10px 0 10px 10px;
}

.mx_CopyableText_copyButton {
Expand All @@ -36,7 +37,8 @@ limitations under the License.
width: 1em;
height: 1em;
cursor: pointer;
margin-left: 20px;
padding-left: 12px;
padding-right: 10px;
display: block;
/* If the copy button is used within a scrollable div, make it stick to the right while scrolling */
position: sticky;
Expand Down
5 changes: 0 additions & 5 deletions res/css/views/settings/tabs/user/_HelpUserSettingsTab.pcss
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,4 @@ limitations under the License.
margin-bottom: $spacing-16;
}
}

/* prevent the access token from overflowing the text box */
div .mx_CopyableText {
overflow: scroll;
}
}

0 comments on commit 9a0e537

Please sign in to comment.