diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index a2a93232..8a294c75 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -16,29 +16,35 @@ If anything in this contributing guide is not clear, don't hesitate to contact W ### Issue Labels -Labels can only be assigned by people with Reporter and higher permissions in the project (see [Members](https://gitlab.com/Isleward/isleward/project_members)). People with rights to assign labels should do so regularly to keep the issues organized. +Labels can only be assigned by people with Reporter and higher permissions in the project (see [Members](https://gitlab.com/Isleward/isleward/project_members)). +People with rights to assign labels should do so regularly to keep the issues organized. The project is using the following, rather self-explanatory, labels: * ~Balance * ~Bug * ~Content +* ~DevOps * ~Draft +* ~Enhancement * ~Feature +* ~Good first issue +* ~Mobile +* ~Mod * ~Modding * ~Polish * ~Refactor -* ~DevOps +* ~Suggestion ### Issue Weights * In addition to labels, issues tagged with the ~Bug label are also assigned Weights. * Bugs are weighed by severity, with 1 being trivial and 5 being severe. -* Weights can assigned by people with Reporter or higher privileges, but this should only be done by @rstan122 unless instructed otherwise. +* Weights can assigned by people with Reporter or higher privileges. ### Confidential Issues * Confidential issues are issues that are visible only to people with Reporter or higher access rights. -* Therefore they are very useful for reporting issues that are very sensitive and that not every user should know about. +* Therefore they are very useful for reporting issues that are very sensitive and that not every user should know about. * To create a confidential issue, simply check the `This issue is confidential and should only be visible to team members with at least Reporter access.` checkbox when creating a new issue. ### Becoming a Reporter @@ -62,7 +68,7 @@ The project is using the following branching structure **release tags** are pointing at the latest commit in `release` at the point of releasing a new version. -**feature branches** are branches in which the actual development happens. The name of a feature branch should always contain the number of the issue that's being addressed (e.g. the `990` in `990-contribuing-rewrite`), be all lowercase with hyphens between words. +**feature branches** are branches in which the actual development happens. The name of a feature branch should always contain the number of the issue that's being addressed (e.g. the `990` in `990-contribuing-rewrite`), and be all lowercase with hyphens between words. ### Using SSH keys @@ -86,14 +92,14 @@ Once you are done working on your contribution, you need to submit a merge reque The merge request should have `master` as its target branch unless arranged otherwise. -You don't need to remove the source branch upon accepting the merge request if the merge request is being made from a fork. +You don't need to remove the source branch upon accepting the merge request if the merge request is being made from a fork. Squishing commits is usually not recommended as it breaks the ability to rebase nicely in certain cases. -Once your merge request is submitted, GitLab CI will run a few tests on it. Should those tests fail, please address the failures. +Once your merge request is submitted, GitLab CI will run a few tests on it. Should those tests fail, please address the failures. A Merge Request with passing tests is ready to be merged. ### Running Isleward * [Windows](https://gitlab.com/Isleward/isleward/wikis/installation-and-usage-(windows)) * [Linux](https://gitlab.com/Isleward/isleward/wikis/installation-and-usage-(linux)) -* [MacOS](https://gitlab.com/Isleward/isleward/wikis/installation-and-usage-(macos)) \ No newline at end of file +* [MacOS](https://gitlab.com/Isleward/isleward/wikis/installation-and-usage-(macos)) diff --git a/src/client/ui/templates/messages/messages.js b/src/client/ui/templates/messages/messages.js index 4b88ac2c..bf43ecf8 100644 --- a/src/client/ui/templates/messages/messages.js +++ b/src/client/ui/templates/messages/messages.js @@ -296,11 +296,13 @@ define([ input.resetKeys(); this.el.removeClass('typing'); + this.el.find('.main').removeClass('hasBorderShadow'); let textbox = this.find('input'); if (show) { this.el.addClass('typing'); + this.el.find('.main').addClass('hasBorderShadow'); if (!config.rememberChatChannel) { this.currentChannel = 'global'; diff --git a/src/client/ui/templates/messages/styles.less b/src/client/ui/templates/messages/styles.less index e3062111..6334ab13 100644 --- a/src/client/ui/templates/messages/styles.less +++ b/src/client/ui/templates/messages/styles.less @@ -222,7 +222,6 @@ } &.typing { - box-shadow: 0 -2px 0 @black, 0 2px 0 @black, 2px 0 0 @black, -2px 0 0 @black; pointer-events: all; .filters { diff --git a/src/client/ui/templates/messages/template.html b/src/client/ui/templates/messages/template.html index 566ee2f3..b64d0ce2 100644 --- a/src/client/ui/templates/messages/template.html +++ b/src/client/ui/templates/messages/template.html @@ -1,20 +1,22 @@
-
info
-
reputation
-
players
-
loot
+
info
+
reputation
+
players
+
loot
-
-
-
- -
-
-
-
-
global
-
party
-
direct
+
+
+
+
+ +
+
+
+
+
global
+
party
+
direct
+
diff --git a/src/client/ui/templates/messages/tplTab.html b/src/client/ui/templates/messages/tplTab.html index 606fff75..5c74e14a 100644 --- a/src/client/ui/templates/messages/tplTab.html +++ b/src/client/ui/templates/messages/tplTab.html @@ -1 +1 @@ -
\ No newline at end of file +
\ No newline at end of file diff --git a/src/client/ui/templates/spells/styles.less b/src/client/ui/templates/spells/styles.less index a6cc44fe..2d757538 100644 --- a/src/client/ui/templates/spells/styles.less +++ b/src/client/ui/templates/spells/styles.less @@ -7,7 +7,6 @@ right: 10px; top: 10px; height: (@btnSize + @pad); - box-shadow: 0 -2px 0 @black, 0 2px 0 @black, 2px 0 0 @black, -2px 0 0 @black; .spell { width: @btnSize; diff --git a/src/client/ui/templates/spells/templateSpell.html b/src/client/ui/templates/spells/templateSpell.html index b565f9f9..386c44b7 100644 --- a/src/client/ui/templates/spells/templateSpell.html +++ b/src/client/ui/templates/spells/templateSpell.html @@ -1,4 +1,4 @@ -
+
$HOTKEY$
diff --git a/src/server/components/stats.js b/src/server/components/stats.js index 1b094f5f..c5d589c7 100644 --- a/src/server/components/stats.js +++ b/src/server/components/stats.js @@ -436,7 +436,9 @@ module.exports = { obj.effects.die(); if (this.obj.spellbook) this.obj.spellbook.die(); + obj.destroyed = true; + obj.destructionEvent = 'death'; let deathAnimation = _.getDeepProperty(animations, ['mobs', obj.sheetName, obj.cell, 'death']); if (deathAnimation) { diff --git a/src/server/world/syncer.js b/src/server/world/syncer.js index ea582a3b..79ab5777 100644 --- a/src/server/world/syncer.js +++ b/src/server/world/syncer.js @@ -59,7 +59,7 @@ module.exports = { continue; let destroyed = o.destroyed; - + let sync = null; let syncSelf = null; if (!destroyed) { @@ -69,7 +69,8 @@ module.exports = { } else { sync = { id: o.id, - destroyed: true + destroyed: true, + destructionEvent: o.destructionEvent }; objects.removeObject(o); @@ -112,8 +113,14 @@ module.exports = { } if (destroyed || !canSee) { - if (!canSee) - queueFunction({ id: oId, destroyed: true }, [ p.serverId ]); + if (!canSee) { + queueFunction({ + id: oId, + destroyed: true, + destructionEvent: 'visibility' + }, + [ p.serverId ]); + } p.player.unsee(oId); }