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

Conversation

@shatfield4
Copy link
Collaborator

resolves #327

Changing styles of "workspace not found" and "text preview" modals to match rest of UI.

@shatfield4 shatfield4 linked an issue Nov 2, 2023 that may be closed by this pull request
@review-agent-prime
Copy link

frontend/src/components/WorkspaceChat/index.jsx

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <dialog
      open={true}
      className="dialogStyle"
    >

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <div className="dialogDivStyle">

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <a
      href={paths.home()}
      className="homeLinkStyle"
    >

frontend/src/pages/Admin/Chats/ChatRow/index.jsx

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <div className="dialogDivStyle">

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <button
      onClick={() => hideModal(modalName)}
      type="button"
      className="buttonStyle"
      data-modal-hide="staticModal"
    >

It's recommended to use CSS classes instead of inline styles for better maintainability and readability. Inline styles can become hard to manage in larger codebases.
Create Issue

    <pre className="preStyle">

@timothycarambat timothycarambat merged commit eebafd7 into master Nov 2, 2023
@timothycarambat timothycarambat deleted the 327-replace-old-modals-to-match-ui branch November 2, 2023 23:17
franzbischoff referenced this pull request in franzbischoff/anything-llm Nov 4, 2023
* workspace not found modal updated to match ui

* update text preview modal to match new ui

* Mobile styles

---------

Co-authored-by: timothycarambat <rambat1010@gmail.com>
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* workspace not found modal updated to match ui

* update text preview modal to match new ui

* Mobile styles

---------

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.

Replace old modals to match ui

3 participants