Skip to content

fix: removes native overlay#48

Merged
pandeymangg merged 1 commit intomainfrom
fix/remove-native-overlay
Feb 13, 2026
Merged

fix: removes native overlay#48
pandeymangg merged 1 commit intomainfrom
fix/remove-native-overlay

Conversation

@pandeymangg
Copy link
Contributor

removes native overlay so the webview handles overlay

@pandeymangg pandeymangg requested a review from Dhruwang February 12, 2026 12:02
@sonarqubecloud
Copy link

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Walkthrough

The pull request simplifies the survey web view component by removing dynamic overlay color derivation logic. The component previously used a helper function and memoized styles to compute modal container styling based on overlay colors. These dependencies have been eliminated in favor of static modal container styling. The overlay value is now computed inline using a fallback pattern rather than through memoization. This reduces the complexity of the styling logic and removes unnecessary type imports and helper functions.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: removing native overlay logic to allow the webview to handle overlay instead.
Description check ✅ Passed The description is directly related to the changeset, explaining the intent behind removing native overlay functionality so webview handles it.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pandeymangg pandeymangg added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit ee26ef1 Feb 13, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants