This restores the old behavior of resetting the unread marker on channel change, as that's usually at this point one wants to check for new messages and is also what matches on the server. I feel this is overall more consistent and useful, and also more in line with what other clients do.
- Transition on the search icon was removed, because why was it even
here?!
- A transition was added to the "Play sound" button
- Transition on the Send button is now consistent with the others
This commit applies the following across the whole style:
- `font` statement is now the same as official FontAwesome CSS
- Ensure icons are never italic or bold or that other variants can be
applied
- Ensure font-size and line-height of icons are inherited from parent
- font-family and font-smoothing is now defined only once
A few (mostly positive) side effects from these and related changes:
- Header icons (main menu, context menu and user list) are now
vertically centered!
- Same applies to the Send icon, but it's more subtle there
- Alignment of the footer icons are shifted a tiny bit
- Server window icons are a wee bit bit bigger to match the server name
font-size
- The "Play sound" icon and text are now both 14px (was 14px / 16px)
Before that change, running a release would display this in the console:
```
fatal: Not a git repository (or any of the parent directories): .git
```
Also, this adds strict mode for that file, and make sure `gitCommit` never throws a `ReferenceError`.
Not that it matters (7.9ms->3.8ms), but I figured since I took the time to profile it as a joke I might as well make it a PR. Mobile might have a slight reduction in typing lag from it?
- `client/views/` contains templates whose built version is provided
with each releases
- `.gitignore` doesn't appear in our releases, but I'm not sure if this
comes from npm itself or Travis CI publishing for us. Adding it here
for completeness.
- Surprisingly, `.npmignore` itself is not ignored...
- All other files and directories are for development purposes only
Instead of always sending them back to the very first lobby on the app.
This fixes a bug introduced in recent layout change but does not bring previous behavior back:
When closing active channel, user gets sent to lobby instead of previously visited channel.
On mobile, there is no favicon to toggle, so nothing shows up when a new notification arrives.
This commit changes this by adding a marker on the burger menu icon, visible on all channels
