feat: extract appBar background color and blurred surface to config pipeline by SERDUN · Pull Request #978 · WebTrit/webtrit_phone

SERDUN · 2026-02-27T08:11:48Z

Summary

Add appBarBackgroundColor and appBarBlurredSurface (color, sigmaX, sigmaY) to BasePageConfig and all page configs
Create BlurredSurfaceConfig (freezed DTO) and BlurredSurfaceStyle with full config→style→factory→screen pipeline
Update all 7 main screens (Keypad, Contacts, Favorites, Recents, Conversations, Settings, About) to use configurable blur params
Update page configuration documentation with new common fields

Test plan

Verify keypad screen renders with configured appBarBackgroundColor
Verify appBarBlurredSurface with custom color/sigmaX/sigmaY produces correct blur effect
Verify screens without config values fall back to defaults
Run flutter analyze — no issues

…ipeline Add configurable appBarBackgroundColor and appBarBlurredSurface (color, sigmaX, sigmaY) to the theme config→style pipeline across all 7 main screens, replacing hardcoded values.

Add Common page fields section documenting appBarBackgroundColor and appBarBlurredSurface, update Keypad page example.

Copilot

Pull request overview

This pull request extracts app bar background color and blurred surface configuration from hardcoded values into a configurable pipeline. It introduces two new common fields (appBarBackgroundColor and appBarBlurredSurface) to the BasePageConfig interface, making them available across all page configurations.

Changes:

Created BlurredSurfaceConfig (freezed DTO) and BlurredSurfaceStyle to represent blur configuration and visual styling
Added appBarBackgroundColor and appBarBlurredSurface to BasePageConfig and all implementing page configs
Updated all 7 main screens and their factories to use configurable blur parameters instead of hardcoded values
Modified BlurredSurface widget to accept parameters without defaults
Added documentation for the new common page fields

Reviewed changes

Copilot reviewed 35 out of 35 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
packages/webtrit_appearance_theme/lib/models/common/blurred_surface_config.dart	New freezed model for blur configuration with color, sigmaX, sigmaY
packages/webtrit_appearance_theme/lib/models/common/blurred_surface_config.g.dart	Generated JSON serialization code
packages/webtrit_appearance_theme/lib/models/common/blurred_surface_config.freezed.dart	Generated freezed code for immutable model
packages/webtrit_appearance_theme/lib/models/common/common.dart	Export statement for new config
packages/webtrit_appearance_theme/lib/models/pages/base_page_config.dart	Added appBarBackgroundColor and appBarBlurredSurface getters
packages/webtrit_appearance_theme/lib/models/theme_page_config.dart	Added new fields to all page config classes
packages/webtrit_appearance_theme/lib/models/theme_page_config.g.dart	Generated JSON serialization for all page configs
packages/webtrit_appearance_theme/lib/models/theme_page_config.freezed.dart	Generated freezed code for all page configs
lib/widgets/blurred_surface_style.dart	New style class with merge/lerp support
lib/widgets/blurred_surface.dart	Changed default sigma values from 10 to 0, removed default color logic
lib/theme/extension/blurred_surface_config_extension.dart	Extension to convert config to style
lib/theme/extension/extension.dart	Export for new extension
lib/theme/styles/base_screen_style.dart	Added appBar fields to base style
lib/theme/factory/styles/*_screen_style_factory.dart	All 7 factories updated to wire config to style
lib/features//view/_screen_style.dart	All 7 screen styles updated with new fields
lib/features//view/_screen.dart	All 7 screens updated to use configurable blur
docs/page_configuration.md	Documentation for new common fields

Comments suppressed due to low confidence (1)

lib/widgets/blurred_surface.dart:27

The default color logic was removed from the BlurredSurface widget. Previously, it would fall back to Theme.of(context).canvasColor.withAlpha(150) when no color was provided. Now it directly uses the nullable color, which could result in a transparent overlay when no color is configured. This breaks the fallback behavior and may result in screens having no visible overlay color when the config doesn't specify one.

    return ClipRect(
      child: BackdropFilter(
        filter: ImageFilter.blur(sigmaX: sigmaX, sigmaY: sigmaY),
        child: child ?? Container(color: color),
      ),
    );

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-27T08:17:53Z

 /// scrollable content appears blurred beneath the app bar.
 class BlurredSurface extends StatelessWidget {
-  const BlurredSurface({super.key, this.color, this.sigmaX = 10, this.sigmaY = 10, this.child});
+  const BlurredSurface({super.key, this.color, this.sigmaX = 0, this.sigmaY = 0, this.child});


The default blur sigma values were changed from 10 to 0. When sigmaX and sigmaY are both 0, no blur effect is applied. This is a breaking change that will affect all existing uses of BlurredSurface that relied on the default blur effect. Screens need to explicitly provide non-zero values or the blur effect will be disabled by default.

Suggested change

const BlurredSurface({super.key, this.color, this.sigmaX = 0, this.sigmaY = 0, this.child});

const BlurredSurface({super.key, this.color, this.sigmaX = 10, this.sigmaY = 10, this.child});

Copilot · 2026-02-27T08:17:54Z

+          sigmaX: effectiveStyle?.appBarBlurredSurface?.sigmaX ?? 0,
+          sigmaY: effectiveStyle?.appBarBlurredSurface?.sigmaY ?? 0,


All screens now default to 0 for blur sigma values when no config is provided. This means that without explicit configuration, the blur effect will be disabled across all screens. This is a behavioral breaking change - previously the BlurredSurface widget had default values of 10 for both sigmaX and sigmaY. Consider using a sensible non-zero default (e.g., 10) to maintain backward compatibility and avoid requiring configuration for basic blur functionality.

Suggested change

sigmaX: effectiveStyle?.appBarBlurredSurface?.sigmaX ?? 0,

sigmaY: effectiveStyle?.appBarBlurredSurface?.sigmaY ?? 0,

sigmaX: effectiveStyle?.appBarBlurredSurface?.sigmaX ?? 10,

sigmaY: effectiveStyle?.appBarBlurredSurface?.sigmaY ?? 10,

Copilot · 2026-02-27T08:17:54Z

+  BlurredSurfaceStyle copyWith({Color? color, double? sigmaX, double? sigmaY}) {
+    return BlurredSurfaceStyle(
+      color: color ?? this.color,
+      sigmaX: sigmaX ?? this.sigmaX,
+      sigmaY: sigmaY ?? this.sigmaY,
+    );
+  }


The copyWith method has a flaw: it cannot distinguish between explicitly passing null and not passing a parameter. If you want to set a value to null explicitly, you cannot do so with this implementation. Consider using a pattern with optional parameters wrapped in Option types, or document that null values cannot be explicitly set via copyWith.

Copilot · 2026-02-27T08:17:54Z

+| Key      | Type   | Default | Description                          |
+|----------|--------|---------|--------------------------------------|
+| `color`  | string | `null`  | Overlay color (hex).                 |
+| `sigmaX` | double | `0`     | Horizontal gaussian blur sigma.      |
+| `sigmaY` | double | `0`     | Vertical gaussian blur sigma.        |


The documentation states that the default blur sigma values are 0, but this effectively disables the blur effect entirely. Consider clarifying in the documentation that users must explicitly configure non-zero values for sigmaX and sigmaY to achieve any blur effect, and provide guidance on recommended values (e.g., 10 for a typical frosted-glass effect).

Suggested change

| Key | Type | Default | Description |

|----------|--------|---------|--------------------------------------|

| `color` | string | `null` | Overlay color (hex). |

| `sigmaX` | double | `0` | Horizontal gaussian blur sigma. |

| `sigmaY` | double | `0` | Vertical gaussian blur sigma. |

| Key | Type | Default | Description |

|----------|--------|---------|---------------------------------------------------------------------------------------|

| `color` | string | `null` | Overlay color (hex). |

| `sigmaX` | double | `0` | Horizontal gaussian blur sigma. `0` means no blur; use a non-zero value (e.g. `10`). |

| `sigmaY` | double | `0` | Vertical gaussian blur sigma. `0` means no blur; use a non-zero value (e.g. `10`). |

SERDUN added 2 commits February 26, 2026 19:59

feat: extract appBar background color and blurred surface to config p…

2eb087e

…ipeline Add configurable appBarBackgroundColor and appBarBlurredSurface (color, sigmaX, sigmaY) to the theme config→style pipeline across all 7 main screens, replacing hardcoded values.

feat: update page configuration docs with appBar fields

2354f3c

Add Common page fields section documenting appBarBackgroundColor and appBarBlurredSurface, update Keypad page example.

SERDUN changed the base branch from main to develop February 27, 2026 08:12

SERDUN requested a review from Copilot February 27, 2026 08:13

Copilot started reviewing on behalf of SERDUN February 27, 2026 08:13 View session

Copilot AI reviewed Feb 27, 2026

View reviewed changes

SERDUN added the draft Not ready but can be start to review label Feb 27, 2026

SERDUN closed this Feb 27, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: extract appBar background color and blurred surface to config pipeline#978

feat: extract appBar background color and blurred surface to config pipeline#978
SERDUN wants to merge 2 commits into
developfrom
feature/appbar-blurred-surface-config

SERDUN commented Feb 27, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 27, 2026

Uh oh!

Copilot AI Feb 27, 2026

Uh oh!

Copilot AI Feb 27, 2026

Uh oh!

Copilot AI Feb 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	const BlurredSurface({super.key, this.color, this.sigmaX = 0, this.sigmaY = 0, this.child});
	const BlurredSurface({super.key, this.color, this.sigmaX = 10, this.sigmaY = 10, this.child});

		sigmaX: effectiveStyle?.appBarBlurredSurface?.sigmaX ?? 0,
		sigmaY: effectiveStyle?.appBarBlurredSurface?.sigmaY ?? 0,

Uh oh!

Conversation

SERDUN commented Feb 27, 2026

Summary

Test plan

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants