Skip to content

fix: update appbar decoration to match khalti brand#23

Open
iamnabink wants to merge 2 commits intokhalti:mainfrom
iamnabink:main
Open

fix: update appbar decoration to match khalti brand#23
iamnabink wants to merge 2 commits intokhalti:mainfrom
iamnabink:main

Conversation

@iamnabink
Copy link
Copy Markdown

Currently payment page has white non-visible appbar which needs to be changed to something visible so added default khalti brand color.

iamnabink and others added 2 commits August 15, 2025 11:53
@mrdeephang
Copy link
Copy Markdown

@iamnabink
The PR mentions matching Khalti branding, but no actual brand color is used. The hardcoded Color(0xFFE8F0F7) also caused visibility issues. It’s better to rely on the app’s theme or use an official brand color of Khalti(Color(0xFFDC0019)) with foregroundColor: Colors.white.
So my proposed change is:

  1. //uses khalti brand color
    title: const Text(s_payWithKhalti),
    backgroundColor: Color(0xFFDC0019),
    foregroundColor: Colors.white,
    actions: [
    IconButton(
    onPressed: _reload,
    icon: const Icon(Icons.refresh),
    )
    ],
    elevation: 4,
    ),

  2. //uses your app brand Color
    title: const Text(s_payWithKhalti),
    actions: [
    IconButton(
    onPressed: _reload,
    icon: const Icon(Icons.refresh),
    )
    ],
    elevation: 4,
    ),
    Use the app’s brand color via AppBarTheme defined in MaterialApp instead of hardcoding colors.
    appBarTheme: const AppBarTheme(
    centerTitle: true,
    foregroundColor: Colors.white,
    backgroundColor: primaryColor,
    elevation: 0,
    ),

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