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
HPCC-23007 Files menu: file list, warning #13386
HPCC-23007 Files menu: file list, warning #13386
Conversation
https://track.hpccsystems.com/browse/HPCC-23007 |
@GordonSmith or @miguelvazq please review. I have created a documentation ticket as well should this need to be updated https://track.hpccsystems.com/browse/HPCC-23545 I have attached a screenshot so you can see what it will look like for users. Thanks |
@@ -151,13 +151,15 @@ define([ | |||
|
|||
refreshState: function () { | |||
if (this.exists()) { | |||
this.iconFilter.src = Utility.getImageURL("filter1.png"); | |||
this.iconFilter.style.color = "cornflowerblue"; |
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.
Where did cornflowerblue
come from? Does it match our currently color?
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.
It matched better than the blue color, and it matched our current color.
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.
@miguelvazq you can open a ticket to start relocating these into the new theme file so that they can get referenced by "concept" rather than specific color?
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.
It matched better than the blue color, and it matched our current color.
Couldn't you match exactly using ColorZilla eye dropper tool to make sure we are consistent?
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.
@miguelvazq you can open a ticket to start relocating these into the new theme file so that they can get referenced by "concept" rather than specific color?
@GordonSmith @kunalaswani I will change these in our material-ui theme. And give them our palette treatment in there since the theme can support FA as well.
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.
Code looks good to me?
Filter Icon updated, along with new color change feature and hover to display warning. Signed-off-by: Kunal Aswani <Kunal.Aswani@lexisnexisrisk.com>
0f2fb0c
to
4bac436
Compare
@miguelvazq colors changed, exact match to blue, and icon alert changed to red as to match other elements. Please review. |
Automated Smoketest: ✅
|
Automated Smoketest: ✅
|
Filter Icon updated, along with new color change feature and hover to
display warning.
Signed-off-by: Kunal Aswani Kunal.Aswani@lexisnexisrisk.com
Type of change:
Checklist:
Smoketest:
Testing: