Skip to content

Commit bb941c1

Browse files
committed
fix: address review comments - remove duplicate CSS and fix RTL hack
- Removed duplicate .dropdown-container class definition in index.css (line 577) - Replaced RTL direction hack with proper text-align approach in ModesView.tsx for better accessibility
1 parent 670cd67 commit bb941c1

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

webview-ui/src/components/modes/ModesView.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -670,8 +670,8 @@ const ModesView = ({ onDone }: ModesViewProps) => {
670670
{modeConfig.name}
671671
</span>
672672
<span
673-
className="text-foreground text-xs opacity-70 truncate flex-shrink-0 max-w-[40%] text-right"
674-
style={{ direction: "rtl" }}
673+
className="text-foreground text-xs opacity-70 truncate flex-shrink-0 max-w-[40%]"
674+
style={{ textAlign: "right" }}
675675
title={modeConfig.slug}>
676676
{modeConfig.slug}
677677
</span>

webview-ui/src/index.css

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -570,12 +570,6 @@ input[cmdk-input]:focus {
570570
max-width: 100%;
571571
}
572572

573-
.dropdown-container {
574-
min-width: 0;
575-
flex: 1;
576-
overflow: hidden;
577-
}
578-
579573
/* Settings view specific responsive adjustments */
580574
@container (max-width: 400px) {
581575
/* Reduce button sizes in very narrow containers */

0 commit comments

Comments
 (0)