mirror of
https://github.com/penpot/penpot.git
synced 2026-04-30 21:59:10 +00:00
4902037c7d
* ✨ Add HEX, HSB, and HSL support in the third color tab Relabel the existing HSVA tab to HSBA (the math was already HSB) and add an inline HSB ↔ HSL model toggle inside the tab, matching Figma's color panel. Sliders, gradients, and labels update dynamically per mode; HSL values roundtrip through RGB/HSV so no color-storage changes are needed. Model choice persists across sessions. * 💄 Fix lint errors Signed-off-by: juan-flores077 <toptalent399@gmail.com> * 🐛 Fix Plugin API token application for JS array of strings (#9166) * 🐛 Fix Plugin API token application for JS array of strings Plugin code calling `shape.applyToken(token, ["fill"])` or `token.applyToShapes([rect], ["fill"])` from JavaScript supplies a JS array of strings. The plugin proxies expected a Clojure set of keywords, and two coupled defects made the calls silently no-op (or, with `throwValidationErrors` enabled, throw "check error"): 1. `token-attr-plugin->token-attr` only consulted its alias map when the input was already a keyword. String inputs like "fill" fell through to the identity branch, so the downstream `cto/token-attr?` predicate (which checks against a set of keywords) returned false for every string. Coerce strings to keywords first. 2. The `applyToken` / `applyToShapes` / `applyToSelected` schemas used plain `[:set ...]`, which has no `:decode/json` transformer for JS array → Clojure set coercion. Switch to the registered `[::sm/set ...]` (in `app.common.schema`) which provides the array → set decoder. After the switch, the standard JSON pipeline converts `["fill"]` to `#{"fill"}`, then the inner `[:and ::sm/keyword [:fn token-attr?]]` decodes each element to a keyword and validates it. Also extends the docstring on `token-attr-plugin->token-attr` to make the string-friendly contract explicit, and registers a new `tokens-test` ns under `frontend/test/frontend_tests/plugins/` with six `deftest` blocks covering: - known keywords passing through unchanged - keyword aliases (`:r1` → `:border-radius-top-left`, etc.) - string inputs coerced to keywords (regression for #9162) - `token-attr?` accepting both keyword and string inputs - `token-attr?` rejecting unknown attrs and nil Closes #9162 * 🐛 Fix wrong direction in plugin-name alias tests The added tests in tokens_test.cljs and the new docstring in tokens.cljs described the alias resolution in the wrong direction. The map is {:r1 :border-radius-top-left, …} then map-invert'd, so token-attr-plugin->token-attr maps verbose plugin-side names (:border-radius-top-left) to canonical internal short names (:r1), not the other way around. Inputs already in canonical form (:r1, :fill, "fill", …) pass through unchanged. Flipped the alias-resolution test expectations and the keyword/string-input cases, refreshed the docstring and the regression-coverage comment to match. --------- Co-authored-by: Andrey Antukh <niwi@niwi.nz> * 💄 Fix sucess typo in subscription dialog i18n keys (#9204) Rename subscription.settings.sucess.dialog.{title,footer} to subscription.settings.success.dialog.{title,footer} in en.po and update the three callsites in subscription.cljs. Closes #9203 Signed-off-by: jack-stormentswe <crazycoder131@gmail.com> * 🐛 Fix HSVA → HSBA test rename and Prettier formatting Signed-off-by: juan-flores077 <toptalent399@gmail.com> * 🐛 Fix CI failures and address review feedback for HSB color tab Signed-off-by: juan-flores077 <toptalent399@gmail.com> * 💄 Resolve Conflicts Signed-off-by: juan-flores077 <toptalent399@gmail.com> --------- Signed-off-by: juan-flores077 <toptalent399@gmail.com> Signed-off-by: jack-stormentswe <crazycoder131@gmail.com> Signed-off-by: Andrey Antukh <niwi@niwi.nz> Co-authored-by: Andrey Antukh <niwi@niwi.nz> Co-authored-by: boskodev790 <boskomaljkovic790@outlook.com> Co-authored-by: Jack Storment <88656337+jack-stormentswe@users.noreply.github.com>