Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(platform): Sync on new UI design #8194

Merged
merged 6 commits into from
Sep 27, 2024
Merged

feat(platform): Sync on new UI design #8194

merged 6 commits into from
Sep 27, 2024

Conversation

majdyz
Copy link
Contributor

@majdyz majdyz commented Sep 26, 2024

Background

Design reference: https://www.figma.com/design/Oeu3UE3k134biyWQL2UL9B/Simple-updates-to-UI-before-beta?node-id=35-1060&node-type=canvas&t=LHPSt1U9Nu0dVWbu-0)](https://www.figma.com/design/Oeu3UE3k134biyWQL2UL9B/Simple-updates-to-UI-before-beta?node-id=35-1060&node-type=canvas&t=LHPSt1U9Nu0dVWbu-0

This PR brings the current UI one step closer to the design.

Changes 🏗️

Navigation

  • Avoid redirecting to unimplemented /profile, redirect to home / instead.
  • Fix broken zoom-in zoom-out due to the run button covering it.

Layout

  • Make flow-editor full-screen: Remove border and make the whole screen an editor
  • On block listing: top border for scrolling is added, and vertical padding is removed.

Text

  • Add credits text back.

Navbar

  • Align styling, and reduce excessive shadow & radius.
  • Add button highlight on active page.

Button

  • Make the primary button responsive on a smaller screen.
  • Re-order palette buttons, moved save under undo & redo.
  • Align styling on palette buttons.
  • Hide the tutorial button on a smaller screen.
image image image image

Testing 🔍

Note

Only for the new autogpt platform, currently in autogpt_platform/

  • Create from scratch and execute an agent with at least 3 blocks
  • Import an agent from file upload, and confirm it executes correctly
  • Upload agent to marketplace
  • Import an agent from marketplace and confirm it executes correctly
  • Edit an agent from monitor, and confirm it executes correctly

@majdyz majdyz requested a review from a team as a code owner September 26, 2024 17:33
Copy link

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review

Possible UI Regression
The ControlPanel component has been moved inside the ReactFlow component, which might affect its positioning and visibility. Ensure this change doesn't negatively impact the user interface.

Layout Concern
The flow-container class now sets position to absolute and dimensions to 100vw and 100vh. This might cause layout issues or unexpected behavior in different screen sizes or when used within other components.

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end size/m labels Sep 26, 2024
Copy link

netlify bot commented Sep 26, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 86c0525
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66f59b069a90ad0008374a4e

Copy link

netlify bot commented Sep 26, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 3f77c40
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/66f650671d73e80008b5e223

@github-actions github-actions bot added size/l and removed size/m labels Sep 26, 2024
@majdyz majdyz requested a review from ntindle September 26, 2024 22:59
Copy link
Contributor

@Swiftyos Swiftyos left a comment

Choose a reason for hiding this comment

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

LGTM

@Swiftyos Swiftyos merged commit 1286a1b into master Sep 27, 2024
13 checks passed
@Swiftyos Swiftyos deleted the zamilmajdy/ui-update branch September 27, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants