diff --git a/packages/feedback/src/screenshot/components/CropIcon.tsx b/packages/feedback/src/screenshot/components/CropIcon.tsx index 2f4dcc61ca6a..091179d86004 100644 --- a/packages/feedback/src/screenshot/components/CropIcon.tsx +++ b/packages/feedback/src/screenshot/components/CropIcon.tsx @@ -9,7 +9,7 @@ export default function CropIconFactory({ }: FactoryParams) { return function CropIcon(): VNode { return ( - + +
-
- - {isCropping && ( +
+ + {(!options._experiments.annotations || isCropping) && (
)} -
- - -
+ + +
+ )}
-
@@ -492,40 +491,24 @@ export function ScreenshotEditorFactory({
diff --git a/packages/feedback/src/screenshot/components/ScreenshotInput.css.ts b/packages/feedback/src/screenshot/components/ScreenshotInput.css.ts index 3f78e41a5851..4a9b313cdd6b 100644 --- a/packages/feedback/src/screenshot/components/ScreenshotInput.css.ts +++ b/packages/feedback/src/screenshot/components/ScreenshotInput.css.ts @@ -40,6 +40,13 @@ export function createScreenshotInputStyles(styleNonce?: string): HTMLStyleEleme ); } +.editor__annotation { + z-index: 1; +} +.editor__annotation--active { + z-index: 2; +} + .editor__canvas-container { width: 100%; height: 100%; @@ -55,7 +62,15 @@ export function createScreenshotInputStyles(styleNonce?: string): HTMLStyleEleme } .editor__crop-container { + custor: auto; position: absolute; + z-index: 2; +} +.editor__crop-container--inactive { + z-index: 1; +} +.editor__crop-container--move { + cursor: move; } .editor__crop-btn-group { @@ -65,6 +80,10 @@ export function createScreenshotInputStyles(styleNonce?: string): HTMLStyleEleme background: var(--button-background, var(--background)); width: 175px; position: absolute; + display: none; +} +.editor__crop-btn-group--active { + display: flex; } .editor__crop-corner { @@ -96,20 +115,28 @@ export function createScreenshotInputStyles(styleNonce?: string): HTMLStyleEleme border-top: none; } .editor__tool-container { - padding-top: 10px; + padding-top: 8px; display: flex; justify-content: space-between; } .editor__tool-bar { - height: 30px; display: flex; - gap: 8px; + gap: 8px; } .editor__tool { + display: flex; + padding: 8px 12px; justify-content: center; align-items: center; border: var(--button-border, var(--border)); border-radius: var(--button-border-radius, 6px); + background: var(--button-background, var(--background)); + color: var(--button-foreground, var(--foreground)); +} + +.editor__tool--active { + background: var(--button-primary-background, var(--accent-background)); + color: var(--button-primary-foreground, var(--accent-foreground)); } `;