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
fix : add some code to reset the color in delta mode #3368
fix : add some code to reset the color in delta mode #3368
Conversation
CHANGELOG.md
Outdated
@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/) | |||
|
|||
## [unreleased] (Added 🚀 | Changed | Removed 🗑 | Fixed 🐞 | Chore 👨💻 👩💻) | |||
|
|||
## [1.120.0] - 2023-09-17 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be set during building the release.
## [1.120.0] - 2023-09-17 |
@@ -70,7 +72,10 @@ export class CodeMapBuilding { | |||
|
|||
resetColor() { | |||
this._color = this._defaultColor | |||
this._deltaColor = this._defaultDeltaColor | |||
// this._deltaColor = this._defaultDeltaColor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// this._deltaColor = this._defaultDeltaColor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for not noticing this suggested change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @csu-it-worker , thank you very much for fixing this :-)
I tested the changes and would like to merge it after the changelog is fixed.
@csu-it-worker any update on this? |
yes, I made the suggested changes |
in delta mode, add a filed to record the original color (red or green), and call method "resetColor()" to reset the color when the building loses focus
@csu-it-worker sorry for the silence. I need to adjust a few tests to apply this fix. I will look into this after the angular 16 update is done. Thanks again for the hint on the fix 🙏 |
Fixed via #3426 |
Add some code to reset the color #3348
Closes: #3348
Description
Resolve the bug : Deselected buildings with green/red roof in delta mode do not reset their color roof
Add a filed to record the original color (red or green)
Call method "resetColor()" to reset the color when the building loses focus In delta mode
Screenshots or gifs