Skip to content

feat: Add support for dark mode images and refactor PropertyControl#464

Merged
surajshetty3416 merged 11 commits intodevelopfrom
option-to-set-dark-mode-image
Jan 27, 2026
Merged

feat: Add support for dark mode images and refactor PropertyControl#464
surajshetty3416 merged 11 commits intodevelopfrom
option-to-set-dark-mode-image

Conversation

@surajshetty3416
Copy link
Member

@surajshetty3416 surajshetty3416 commented Jan 22, 2026

  • Support for dark mode images with dynamic source handling.
Screen.Recording.2026-01-27.at.4.12.33.PM.mov
  • Refactored the PropertyControl component into specialised controls for better organisation and maintainability.
    • We now have AttributePropertyControl (for HTML attributes), StylePropertyControl (for style properties) and BasePropertyControl (for general purpose)
    • BasePropertyControl now has "variants" option using which a control can be extended
    • StylePropertyControl internally uses variants to show state style options in the dropdown
    • Image src propertyControl uses variant to show "Dark Mode" option

- Added support for variant. Using this we can upload dark mode of an image
- Split PropertyControl into BasePropertyControl, StylePropertyControl, AttributePropertyControl for focused usage
- Replace usages of PropertyControl with appropriate new controls
@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

❌ Patch coverage is 22.72727% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.56%. Comparing base (49264dc) to head (9edb31b).
⚠️ Report is 12 commits behind head on develop.

Files with missing lines Patch % Lines
...ilder/builder/doctype/builder_page/builder_page.py 22.72% 17 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #464      +/-   ##
===========================================
- Coverage    52.90%   52.56%   -0.35%     
===========================================
  Files           28       28              
  Lines         2202     2222      +20     
===========================================
+ Hits          1165     1168       +3     
- Misses        1037     1054      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@surajshetty3416 surajshetty3416 force-pushed the option-to-set-dark-mode-image branch from 250c80c to 9edb31b Compare January 27, 2026 09:31
@surajshetty3416 surajshetty3416 marked this pull request as ready for review January 27, 2026 10:52
@surajshetty3416 surajshetty3416 merged commit 0f71836 into develop Jan 27, 2026
4 of 6 checks passed
@github-actions
Copy link

🎉 This PR is included in version 1.22.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

surajshetty3416 added a commit that referenced this pull request Feb 7, 2026
After #464, property controls stopped accepting values as boolean and number and used to auto convert all values to string, which is why components like TabButton stopped working as expected.
mergify bot pushed a commit that referenced this pull request Feb 7, 2026
After #464, property controls stopped accepting values as boolean and number and used to auto convert all values to string, which is why components like TabButton stopped working as expected.

(cherry picked from commit 37a92c2)

# Conflicts:
#	frontend/src/components/Controls/BasePropertyControl.vue
#	frontend/src/components/Controls/PropertyControlInput.vue
#	frontend/src/components/Controls/TabButtons.vue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant