-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Actions modal and icon UI update #14856
base: master
Are you sure you want to change the base?
Conversation
Modal size and layout - increase the size of the modal header (and others) - increased modal size - changed the grid to 3 columns rather than 2 - increase the size of the label text - moved actions to the top and apps to the bottom Items: - increased the size of the automation labels - added a background color to action icons - increase the size of all icons - changed the color of action icons to gray 900 item cards: - updated the spacing within the item cards - changed border color of items disabled items: - added opacity to the disabled actions
- added background to trigger - orange - added background color to actions - indigo
- changed backend log icon - added static values for trigger icon and action icons - added a background to external app actions - changed layout of labels + fields - resized automation cards - removed automation label. Made Automation title more prominent. - creation automation modal layout changed - removed label for name field - removed help text - removed warning - added background to icons - changed item size - added placeholder to name input
- improved inline alert title spacing - added a background color to the modal component - it was grey on light - changed static color of triggers - yellow -
- the grey on grey has bugged me for eternity. I also like how it looks in darkest mode too - the contrast is nice.
- updated tag component to make it stand out more and add more color to the UI. Most of the time, the tag component is used to inform the user if a feature is paid. This work emphasizes this.
QA Wolf here! As you write new code it's important that your test coverage is keeping up. |
I initially removed the warning from the new automation model. @PClmnt felt it was needed. I feel that if it is needed, this is not the place to show it - considering it's the first thing people see, and the action (publish) is the last thing users do. Good spot with the looping icon. I wanted to create a 3rd category - Logic or something and provide a different color for them. If I was to make another change - I would change the shape of the trigger icon to a hexagon or something to make it really stand out - or the budibase logo arrow. Branching arrows
Also... if we do add back the arrows, I'd probably add a more icon and place the delete, left, and right functionality in there. |
https://www.figma.com/design/LZJRE6DsiGGCEjB7aqFYGx/Automations?node-id=704-5340&t=0pJ9dqYNWcX9ZPjL-1 Joe and had a call earlier with version iterations on the automation branching. As you'll see I've amended the design to include Joe's latest additions as well as some of my own. These include having 'First check this branch', then 'then check this branch' at the top of the nodes. In addition, we've removed the info text and placed the branch movement buttons in a submenu with contextual labels, that way the user understands what they mean. Let me know your thoughts! |
- updated looping icon (cyan + black) - updated looping label type - to p - removed label margin
- spacing (between delete and chevron icons) was different between looping and actions
- handle religned to prevent card from growing - spacing improved with modal cards - automations closed by default - background and border added to header to improve structure
- changed color of help icon - added hover color to drag container
Description
I updated the UI in several places, primarily the Automations section, to improve the design and experience.
I made styling changes that were not attached to our design system - specifically, the yellow and indigo within automations. These colors make the UI "pop." Happy to discuss this in more detail.
@deanhannigan I removed the left and right arrows on branches—I don't think they're needed. We can add them back if necessary.
Screenshots
Automations
Darkest
Light
New Automation Modal
Darkest
Light
New Automation Action Modal
It would be beneficial to introduce a third category - Logic to this modal. I would also change Condition to "Continue If"
Darkest
Light
New Automation Card
Darkest
Tag
Darkest
Light
CTA tag
Non-CTA tag
Alignment issue
Add a small description in layman's terms of what this PR achieves. This will be used in the release notes.
I believe it improves the overall UX/UI