这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@shatfield4
Copy link
Collaborator

@shatfield4 shatfield4 commented Feb 27, 2024

Pull Request Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #809

What is in this change?

Describe the changes in this PR that are impactful to the repo.

  • Implement new sidebar designs per internal designs
  • Match borders and border radius to match in the rest of the application

Additional Information

Add any other context about the Pull Request here that was not captured above.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@shatfield4 shatfield4 self-assigned this Feb 27, 2024
@shatfield4 shatfield4 linked an issue Feb 27, 2024 that may be closed by this pull request
@shatfield4 shatfield4 changed the title WIP main sidebar designs [STYLE] Implement new side bar menus Feb 27, 2024
@shatfield4 shatfield4 marked this pull request as ready for review February 27, 2024 05:34
Fix conditional hook render
Copy link
Member

@timothycarambat timothycarambat left a comment

Choose a reason for hiding this comment

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

See comment for permission fix. Always validate changes in single and multi-user mode

@timothycarambat timothycarambat removed their assignment Mar 7, 2024
@timothycarambat timothycarambat merged commit 3cb8a80 into master Mar 7, 2024
@timothycarambat timothycarambat deleted the 809-style-implement-new-side-bar-menus branch March 7, 2024 01:22
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* WIP main sidebar designs

* hover states and active states for main sidebar

* main and settings sidebar UI updates & improve performance using Link instead of <a>

* update borders to match rest of UI in all pages

* update borders of all containers to match rest of UI

* remove unneeded conditional

* small changes to UI to conform to designs
Fix conditional hook render

* add check for roles to SettingsButton component and hide button from footer when in mobile

* typo

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
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.

[STYLE] Implement new side bar menus

3 participants