Skip to content

fix client state inside of rxcond#5340

Merged
adhami3310 merged 2 commits intomainfrom
fix-client-state-inside-of-rxcond
May 26, 2025
Merged

fix client state inside of rxcond#5340
adhami3310 merged 2 commits intomainfrom
fix-client-state-inside-of-rxcond

Conversation

@adhami3310
Copy link
Copy Markdown
Member

misbehaving code:

import reflex as rx
from reflex.experimental.client_state import ClientStateVar

app = rx.App()

SecretNameClientState = ClientStateVar.create("secret_name", default="")
RebootAppCheckboxClientState = ClientStateVar.create("reboot_app", default=False)


@app.add_page
def index():
    return rx.vstack(
        rx.input(
            default_value=SecretNameClientState.value,
            on_change=SecretNameClientState.set_value,
        ),
        rx.checkbox(
            checked=RebootAppCheckboxClientState.value,
            on_change=RebootAppCheckboxClientState.set_value,
        ),
        rx.cond(
            RebootAppCheckboxClientState.value,
            rx.button("A ", f"Name: {SecretNameClientState.value}"),
            rx.button(f"Name: {SecretNameClientState.value}"),
        ),
    )

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented May 26, 2025

CodSpeed Performance Report

Merging #5340 will not alter performance

Comparing fix-client-state-inside-of-rxcond (83b16fc) with main (859e8f8)

Summary

✅ 8 untouched benchmarks

@adhami3310 adhami3310 merged commit c849cd8 into main May 26, 2025
42 checks passed
@adhami3310 adhami3310 deleted the fix-client-state-inside-of-rxcond branch May 26, 2025 18:58
adhami3310 added a commit that referenced this pull request May 26, 2025
* fix client state inside of rxcond

* ruff
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