Skip to content

Commit

Permalink
extensions_ui: Don't block scrolling on disabled extension cards (#22189
Browse files Browse the repository at this point in the history
)

This PR fixes an issue where disabled extension cards would block scroll
events when the mouse was over them.

Instead of using `.occlude`—which captures _all_ mouse events—we use
`.block_mouse_down` to just block mouse down events (to prevent clicking
the buttons on the disabled card). We also set the cursor style to the
default, as it was still switching to the pointer when hovered over top
of one of the buttons.

Release Notes:

- Fixed disabled extensions blocking scroll events in the extensions
list.
  • Loading branch information
maxdeviant authored Dec 18, 2024
1 parent 4a6f071 commit 670ade9
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion crates/extensions_ui/src/components/extension_card.rs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ impl RenderOnce for ExtensionCard {
.absolute()
.top_0()
.left_0()
.occlude()
.block_mouse_down()
.cursor_default()
.size_full()
.items_center()
.justify_center()
Expand Down

0 comments on commit 670ade9

Please sign in to comment.