Browse Source

style: media query for hiding shortcuts for mobile view (#2667)

Co-authored-by: Lipis <lipiridis@gmail.com>
pull/2674/head
Harshil Parmar 5 years ago committed by GitHub
parent
commit
8091ac6c08
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 24
      src/components/ContextMenu.scss
  2. 4
      src/components/ContextMenu.tsx

24
src/components/ContextMenu.scss

@ -1,4 +1,4 @@ @@ -1,4 +1,4 @@
@import "open-color/open-color.scss";
@import "../css/_variables";
.excalidraw {
.context-menu {
@ -42,16 +42,16 @@ @@ -42,16 +42,16 @@
}
&.dangerous {
div:nth-child(1) {
.context-menu-option__label {
color: $oc-red-7;
}
}
div:nth-child(1) {
.context-menu-option__label {
justify-self: start;
margin-inline-end: 20px;
}
div:nth-child(2) {
.context-menu-option__shortcut {
justify-self: end;
opacity: 0.6;
font-size: 0.7rem;
@ -63,7 +63,7 @@ @@ -63,7 +63,7 @@
background-color: var(--select-highlight-color);
&.dangerous {
div:nth-child(1) {
.context-menu-option__label {
color: var(--popup-background-color);
}
background-color: $oc-red-6;
@ -73,4 +73,18 @@ @@ -73,4 +73,18 @@
.context-menu-option:focus {
z-index: 1;
}
@media #{$media-query} {
.context-menu-option {
display: block;
.context-menu-option__label {
margin-inline-end: 0;
}
.context-menu-option__shortcut {
display: none;
}
}
}
}

4
src/components/ContextMenu.tsx

@ -51,8 +51,8 @@ const ContextMenu = ({ options, onCloseRequest, top, left }: Props) => { @@ -51,8 +51,8 @@ const ContextMenu = ({ options, onCloseRequest, top, left }: Props) => {
${checked ? "checkmark" : ""}`}
onClick={action}
>
<div>{label}</div>
<div>
<div className="context-menu-option__label">{label}</div>
<div className="context-menu-option__shortcut">
{shortcutName
? getShortcutFromShortcutName(shortcutName)
: ""}

Loading…
Cancel
Save