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

ui(footer): improve label and colors of websocket indicator #677

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

theborakompanioni
Copy link
Collaborator

@theborakompanioni theborakompanioni commented Oct 13, 2023

Before this commit, the label on the websocket indicator just stated "connected" or "disconnected", which caused some confusion.

After this commit, it will be labelled "Websocket conntected" or "Websocket disconnected". Also, if the websocket is disconnected, the icon is coloured gray instead of red, as Jam will work normally without a stable websocket connection (UI responsiveness will be slower, but is functioning).

📸 Before

📸 After


@theborakompanioni theborakompanioni self-assigned this Oct 13, 2023
@theborakompanioni theborakompanioni added the UI/UX Issue related to cosmetics, design, or user experience label Oct 13, 2023
@theborakompanioni theborakompanioni force-pushed the chore/websocket-connection-label branch from 1a86f40 to e24b963 Compare October 13, 2023 14:07
@theborakompanioni theborakompanioni force-pushed the chore/websocket-connection-label branch from e24b963 to c93749b Compare October 13, 2023 19:57
@theborakompanioni theborakompanioni marked this pull request as ready for review October 13, 2023 19:59
Copy link

@editwentyone editwentyone left a comment

Choose a reason for hiding this comment

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

as expected, noize

@theborakompanioni theborakompanioni merged commit 2bed51e into master Oct 14, 2023
3 checks passed
@theborakompanioni theborakompanioni deleted the chore/websocket-connection-label branch October 14, 2023 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX Issue related to cosmetics, design, or user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants