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

Do not display incorrect nick when switching to a non connected network #252

Merged
merged 1 commit into from
Apr 30, 2016

Conversation

xPaw
Copy link
Member

@xPaw xPaw commented Apr 10, 2016

Previously it only called setNick if the network had a nick, so it would keep displaying the nick from the network you switched from, this PR fixes that.

  • It also removes the magic 31 margin number, and calculates the margin correctly.
  • Moves : to be appended with CSS.
  • Hides #nick panel if the network has no nick (e.g. while lauching lounge or connecting)

@xPaw xPaw added Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors. Type: Feature Tickets that describe a desired feature or PRs that add them to the project. labels Apr 10, 2016
@maxpoulin64
Copy link
Member

Whoops, that one went unnoticed for a while. 👍

This seems pretty minor to me, feel free to merge if you feel the same.

@astorije
Copy link
Member

👍

@astorije astorije merged commit d73c6d0 into master Apr 30, 2016
@astorije astorije deleted the xpaw/setnick branch April 30, 2016 23:40
@astorije astorije added this to the ★ Next Release milestone May 15, 2016
matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
Do not display incorrect nick when switching to a non connected network
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issues that report and PRs that solve any defects that cause unexpected behaviors. Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants