Fix pink color selection by adding intermediate hue gradient stops#87
Draft
Fix pink color selection by adding intermediate hue gradient stops#87
Conversation
Co-authored-by: nohli <43643339+nohli@users.noreply.github.com>
Co-authored-by: nohli <43643339+nohli@users.noreply.github.com>
Co-authored-by: nohli <43643339+nohli@users.noreply.github.com>
Co-authored-by: nohli <43643339+nohli@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix missing pink color option in color palette
Fix pink color selection by adding intermediate hue gradient stops
Nov 26, 2025
There was a problem hiding this comment.
Pull request overview
This PR fixes a color selection issue where pink colors (hue 300-360°) were difficult to select in color pickers due to insufficient gradient granularity. The fix increases hue gradient stops from 7 to 13, providing stops every 30° instead of 60°.
Key Changes:
- Added 6 new intermediate hue color stops at 30°, 90°, 150°, 210°, 270°, and 330° (pink)
- Updated gradient stops with mathematically accurate RGB values for each hue angle
- Applied changes consistently across all 5 picker components
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
lib/src/pickers/wheel_picker.dart |
Added 6 new hue color stops to the sweep gradient used in the wheel picker |
lib/src/pickers/palette_value_picker.dart |
Added 6 new hue color stops to the hardcoded hueColors list |
lib/src/pickers/palette_saturation_picker.dart |
Added 6 new hue color stops to the hardcoded hueColors list |
lib/src/pickers/palette_hue_picker.dart |
Added 6 new hue color stops using dynamic color generation with withHue() method |
lib/src/pickers/hsv_picker.dart |
Added 6 new hue color stops using dynamic color generation with withHue() method |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pink colors (hue 300-360°) were difficult to select because hue gradients had only 7 color stops, compressing the magenta-to-red transition into 1/6 of the slider.
Changes
PaletteHuePicker,HSVPicker,PaletteSaturationPicker,PaletteValuePicker,WheelPickerWith 13 evenly distributed stops, gradient positions align perfectly with hue values (position N/12 = hue N×30/360), enabling precise selection across the full hue spectrum.
Before: 7 stops → pink range compressed into last 16.67% of slider
After: 13 stops → pink explicitly represented at position 11/12 (91.67%)
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.