Skip to content

Remove the success, warning and alert colors from Organizations' appearance#13035

Merged
alecslupu merged 1 commit intodevelopfrom
change/colors-success-warning-alert
Jul 4, 2024
Merged

Remove the success, warning and alert colors from Organizations' appearance#13035
alecslupu merged 1 commit intodevelopfrom
change/colors-success-warning-alert

Conversation

@andreslucena
Copy link
Copy Markdown
Member

🎩 What? Why?

While discussing with @carolromero about what colors should we have in Organizations' appearance in admin panel for #13034, we agreed that having the "success", "alert" and "warning" colors customization for the organizations is overkill.

We didn't need to change this setting in almost 5 years of having this feature in Decidim, and we also believe that the "semaphore" is international and widely used. Just in case we asked to the Oracle (aka ChatGPT):

The colors of the semaphor (green, red, yellow) are international?

ChatGPT:
Yes, the colors used in traffic signals (green, red, and yellow) are standardized and recognized internationally. This standardization helps ensure consistency and safety in traffic management across different countries and regions.

  • Red: universally indicates "stop."
  • Green: universally indicates "go."
  • Yellow (or amber): universally indicates "caution" or "prepare to stop."

This color scheme has been adopted by most countries worldwide, facilitating a common understanding of traffic signals among drivers and pedestrians, regardless of their location. The standardization is guided by international agreements and recommendations, such as those from the Vienna Convention on Road Signs and Signals, which many countries adhere to.

And how about using these colors for "success", "warning" and "alert"?

ChatGPT:

The use of colors to indicate "success," "warning," and "alert" is also widely standardized and recognized in various fields, including user interface design, safety signage, and communication systems. The colors typically used are:

  • Green: Indicates "success" or "okay." It conveys a positive outcome, such as a completed task or a correct operation.
  • Yellow (or Amber): Indicates "warning" or "caution." It signals that attention is needed but not necessarily an immediate threat. It often suggests that users should be aware of potential issues.
  • Red: Indicates "alert," "error," or "danger." It denotes a critical issue that requires immediate attention or action.

These color associations are not only intuitive but also supported by psychological studies on color perception, making them effective for conveying messages quickly and clearly across different cultures and contexts. This standardization is commonly seen in software interfaces, dashboard indicators, health and safety signs, and various other applications.

📌 Related Issues

  • Related to Adapt image (CSS) for every tenant #4696 - seems like the introduction was because of me, as I've blindly added by copy pasting the variables that we had in the SCSS colors definition 🤦🏽

Testing

  1. Go to http://localhost:3000/admin/organization/appearance/edit
  2. Scroll to the colors section
  3. See that you don't have the "success", "warning" and "alert" colors anymore

📷 Screenshots

Screenshot of the colors section in organizations' Appearance admin form

♥️ Thank you!

@andreslucena andreslucena added the type: change PRs that implement a change for an existing feature label Jun 21, 2024
@andreslucena andreslucena added type: removal PRs that implement a removal of a functionality or code and removed type: change PRs that implement a change for an existing feature labels Jun 21, 2024
@andreslucena andreslucena added this to the 0.29.0 milestone Jun 28, 2024
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: admin module: core module: system type: removal PRs that implement a removal of a functionality or code

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants