Skip to content

Remove image zoom in how reflex works doc#6276

Merged
masenf merged 1 commit intomainfrom
carlos/use-pcweb-component
Apr 2, 2026
Merged

Remove image zoom in how reflex works doc#6276
masenf merged 1 commit intomainfrom
carlos/use-pcweb-component

Conversation

@carlosabadia
Copy link
Copy Markdown
Contributor

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 2, 2026

Greptile Summary

This PR removes the reflex_image_zoom third-party component from the "How Reflex Works" documentation page and replaces the zoom interaction with a simple rx.el.a link that opens the architecture diagram image directly in a new browser tab.

  • Removes the python exec block that imported image_zoom from reflex_image_zoom
  • Replaces image_zoom(rx.image(...)) with rx.el.a(rx.image(...), href="...", target="_blank"), correctly using rx.el.a (unstyled) to avoid unwanted link styling
  • The replacement is functionally equivalent for users who want to see the full image, and eliminates a third-party dependency from the docs page

Confidence Score: 5/5

  • Safe to merge — a straightforward documentation change with no logic or backend impact
  • The change is limited to a single documentation markdown file, removes a third-party dependency cleanly, and the replacement correctly uses rx.el.a (per the team's coding conventions). No logic errors, regressions, or breaking changes are introduced.
  • No files require special attention

Important Files Changed

Filename Overview
reflex/docs/advanced_onboarding/how-reflex-works.md Removes reflex_image_zoom dependency and replaces it with a plain clickable rx.el.a link opening the architecture image in a new tab

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User visits How Reflex Works doc] --> B[Architecture diagram image rendered]
    B --> C{Before PR}
    B --> D{After PR}
    C --> E[image_zoom wrapper\nfrom reflex_image_zoom\nAllows in-page zoom]
    D --> F[rx.el.a wrapper\nOpens full image in new tab\ntarget=_blank]
Loading

Reviews (1): Last reviewed commit: "Remove image zoom in how reflex works do..." | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 2, 2026

Merging this PR will not alter performance

✅ 8 untouched benchmarks


Comparing carlos/use-pcweb-component (8035e3c) with main (980a97d)

Open in CodSpeed

@masenf masenf merged commit 86afe3d into main Apr 2, 2026
6 checks passed
@masenf masenf deleted the carlos/use-pcweb-component branch April 2, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants