-
Notifications
You must be signed in to change notification settings - Fork 483
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
Add color picker for Design Mode icons #7369
Conversation
Hey Josh, I'd be happy to look at this, however I'm under the gun to finish something this afternoon and on vacation tomorrow, so I may not be able to give it a thorough look until Monday morning. |
var iconColorPicker; | ||
var canonicalImage = element.getAttribute('data-canonical-image-url'); | ||
if (ICON_PREFIX_REGEX.test(canonicalImage)) { | ||
|
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.
nit: extra line
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.
same in other jsx files
@@ -180,13 +180,14 @@ designMode.onPropertyChange = function(element, name, value) { | |||
* @param value {string} An icon identifier of the format "icon://fa-icon-name". | |||
* @return {string} | |||
*/ | |||
function renderIconToString(value) { | |||
function renderIconToString(value, element) { |
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.
add @param for element
couple comments, but otherwise lgtm :) |
Conflicts: apps/src/applab/designElements/button.jsx
8ac31ec
to
f4526fa
Compare
Add color picker for Design Mode icons
26954f8 Merge pull request #7369 from code-dot-org/icon-color-picker (Josh Lory) 53cfb6e add column in cdo-languages for which languages are supported on hoc.com (Continuous Integration) 7ca9bc8 Merge pull request #7371 from code-dot-org/use-color-js (Brad Buchanan) 3761f86 fix path to unbreak build (Brent Van Minnen) f4526fa Merge branch 'staging' into icon-color-picker (Josh Lory) ba63a4d Code review feedback (Josh Lory) 42b176a Merge pull request #7367 from code-dot-org/generate-color-js (Brad Buchanan)
Users can now choose a fill color for Design Mode icons: