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

Remove all emojis (except for the operating system emojis) #88

Merged
merged 3 commits into from
Apr 8, 2022

Conversation

DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Apr 8, 2022

This pull request removes all emojis from job names, except for the operating system emojis (the Linux penguin, the macOS Apple logo, etc.).

Motivation

I think that the operating system emojis are very useful. They let you easily distinguish between different operating systems with a quick glance.

However, in my opinion, the other emojis are not very helpful. For example, the test tube emoji (🧪) is redundant, because:

  1. All of the jobs that have 🧪 in the name also have a name that starts with test.
  2. All of the jobs that have 🧪 in the name are in the Test group.

The downside of having these extra emojis at the beginning of job names is that they take up valuable horizontal real estate. Now that we have four top-level groups, all jobs are located in one of those four groups. When you click on a group, a pop-up box appears that shows all of the jobs in that group. Inside that pop-up box, there is not a lot of space, so job names are truncated1. So, I want to remove these unnecessary emojis from the beginnings of the job names, so that more of the actual job name can be displayed.

Footnotes

  1. You can mouse over a job name, and the alt-text that displays will show the full job name. But this is annoying and takes more time.

@DilumAluthge DilumAluthge merged commit f5f7815 into main Apr 8, 2022
@DilumAluthge DilumAluthge deleted the dpa/emoji branch April 8, 2022 06:31
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.

1 participant