Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI Redesign - Mobile - Trusted Senders #231

Closed
wants to merge 1 commit into from
Closed

UI Redesign - Mobile - Trusted Senders #231

wants to merge 1 commit into from

Conversation

Jainakin
Copy link

@Jainakin Jainakin commented Oct 5, 2022

- What I did
Created a new UI design for the trusted senders screen and a remove trusted sender dialog which closes #222

- How I did it
I created a new screen for the trusted senders screen inside the trusted_senders_screen folder

- How to verify it
add the code to the welcome screen and you should be able to click the floating action button to view the new screen

floatingActionButton: FloatingActionButton(
    backgroundColor: const Color(0xFFF05E37),
    child: const ImageIcon(
        AssetImage("assets/images/add_icon.png"),
        color: Colors.white,
        size: 16.0,
    ),
    onPressed: () {
        showModalBottomSheet<dynamic>(
            isScrollControlled: true,
            shape: RoundedRectangleBorder(
                borderRadius: BorderRadius.circular(20.0),
            ),
            context: context,
            builder: (BuildContext context) {
                return TrustedSendersScreen();
            },
        );
    },
),

- Description for the changelog
Added Trusted Senders Screen and Dialog

@Jainakin
Copy link
Author

Jainakin commented Oct 5, 2022

Hey @ksanty, I have completed my assigned task. Can you please check it and give me feedback for corrections if any.
Thank You

@tinashe404
Copy link
Member

We need your work to be going to the redesign branch, not the trunk branch.

@Jainakin Jainakin closed this Oct 6, 2022
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.

UI Redesign - Mobile - Trusted Senders
2 participants