History

xcolor/rgbcolor conversion

SamGondelman committed 8 years ago

Commit 16ad9cb1 parent 992952962 files · +593 −917Browse files

Showing the changes to interface/src/scripting/SelectionScriptingInterface.cpp (1 of 62 files). Show all

Unified · Split

interface/src/scripting/SelectionScriptingInterface.cpp+13−12
@@ -421,7 +421,7 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 420 unmodified lines
421421 auto colorVariant = properties["outlineUnoccludedColor"];
422422 if (colorVariant.isValid()) {
423423 bool isValid;
424− auto color = xColorFromVariant(colorVariant, isValid);
424+ auto color = vec3FromVariant(colorVariant, isValid);
425425 if (isValid) {
426426 _style._outlineUnoccluded.color = toGlm(color);
427427 }
@@ -429,7 +429,7 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 1 unmodified line
429429 colorVariant = properties["outlineOccludedColor"];
430430 if (colorVariant.isValid()) {
431431 bool isValid;
432− auto color = xColorFromVariant(colorVariant, isValid);
432+ auto color = vec3FromVariant(colorVariant, isValid);
433433 if (isValid) {
434434 _style._outlineOccluded.color = toGlm(color);
435435 }
@@ -437,7 +437,7 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 1 unmodified line
437437 colorVariant = properties["fillUnoccludedColor"];
438438 if (colorVariant.isValid()) {
439439 bool isValid;
440− auto color = xColorFromVariant(colorVariant, isValid);
440+ auto color = vec3FromVariant(colorVariant, isValid);
441441 if (isValid) {
442442 _style._fillUnoccluded.color = toGlm(color);
443443 }
@@ -445,7 +445,7 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 1 unmodified line
445445 colorVariant = properties["fillOccludedColor"];
446446 if (colorVariant.isValid()) {
447447 bool isValid;
448− auto color = xColorFromVariant(colorVariant, isValid);
448+ auto color = vec3FromVariant(colorVariant, isValid);
449449 if (isValid) {
450450 _style._fillOccluded.color = toGlm(color);
451451 }
@@ -482,10 +482,10 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 30 unmodified lines
482482
483483 /**jsdoc
484484 * @typedef {object} Selection.HighlightStyle
485− * @property {Color} outlineUnoccludedColor - Color of the specified highlight region.
486− * @property {Color} outlineOccludedColor - ""
487− * @property {Color} fillUnoccludedColor- ""
488− * @property {Color} fillOccludedColor- ""
485+ * @property {Vec3Color} outlineUnoccludedColor - Color of the specified highlight region.
486+ * @property {Vec3Color} outlineOccludedColor - ""
487+ * @property {Vec3Color} fillUnoccludedColor- ""
488+ * @property {Vec3Color} fillOccludedColor- ""
489489 * @property {number} outlineUnoccludedAlpha - Alpha value ranging from <code>0.0</code> (not visible) to <code>1.0</code>
490490 * (fully opaque) for the specified highlight region.
491491 * @property {number} outlineOccludedAlpha - ""
@@ -497,10 +497,11 @@ bool SelectionHighlightStyle::fromVariantMap(const QVariantMap& properties) { · 5 unmodified lines
497497 QVariantMap SelectionHighlightStyle::toVariantMap() const {
498498 QVariantMap properties;
499499
500− properties["outlineUnoccludedColor"] = xColorToVariant(xColorFromGlm(_style._outlineUnoccluded.color));
501− properties["outlineOccludedColor"] = xColorToVariant(xColorFromGlm(_style._outlineOccluded.color));
502− properties["fillUnoccludedColor"] = xColorToVariant(xColorFromGlm(_style._fillUnoccluded.color));
503− properties["fillOccludedColor"] = xColorToVariant(xColorFromGlm(_style._fillOccluded.color));
500+ const float MAX_COLOR = 255.0f;
501+ properties["outlineUnoccludedColor"] = vec3ToVariant(_style._outlineUnoccluded.color * MAX_COLOR);
502+ properties["outlineOccludedColor"] = vec3ToVariant(_style._outlineOccluded.color * MAX_COLOR);
503+ properties["fillUnoccludedColor"] = vec3ToVariant(_style._fillUnoccluded.color * MAX_COLOR);
504+ properties["fillOccludedColor"] = vec3ToVariant(_style._fillOccluded.color * MAX_COLOR);
504505
505506 properties["outlineUnoccludedAlpha"] = _style._outlineUnoccluded.alpha;
506507 properties["outlineOccludedAlpha"] = _style._outlineOccluded.alpha;