Skip to content

Lendemor/templates radix sidebar#2705

Merged
picklelo merged 10 commits intomainfrom
lendemor/templates_radix_sidebar
Feb 28, 2024
Merged

Lendemor/templates radix sidebar#2705
picklelo merged 10 commits intomainfrom
lendemor/templates_radix_sidebar

Conversation

@Lendemor
Copy link
Copy Markdown
Contributor

@Lendemor Lendemor commented Feb 23, 2024

No description provided.

@Lendemor Lendemor marked this pull request as ready for review February 23, 2024 22:04
Copy link
Copy Markdown
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Menu button seems a little too big but overall good change

image

Copy link
Copy Markdown
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually can you fix the broken links in README.md as well?

@Lendemor Lendemor requested a review from masenf February 25, 2024 23:02
masenf
masenf previously approved these changes Feb 27, 2024
Comment thread reflex/.templates/apps/sidebar/code/styles.py Outdated
Comment thread reflex/.templates/apps/sidebar/code/styles.py Outdated
picklelo
picklelo previously approved these changes Feb 28, 2024
Comment thread reflex/.templates/apps/sidebar/code/pages/settings.py Outdated
@picklelo picklelo force-pushed the lendemor/templates_radix_sidebar branch from 0d0a279 to 68f5980 Compare February 28, 2024 22:48
@picklelo picklelo merged commit 3bcd067 into main Feb 28, 2024
@masenf masenf deleted the lendemor/templates_radix_sidebar branch February 29, 2024 21:48
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.

3 participants