CCExtractor / taskwarrior-flutter

This is the repo of mobile app of taskwarrior

Home Page:https://play.google.com/store/apps/details?id=com.ccextractor.taskwarriorflutter

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Enhancement: UI fixes in alert dialogs

its-me-abhishek opened this issue · comments

Describe your issue

Currently, the alert dialogs throughout the app are not consistent with the app's themes and colors.
To make them more visually appealing and cohesive, we can introduce a consistent style, similar to the one in Add Task dialog box. This way, users will have a more appealing experience across the app.

Steps to reproduce

  1. Save or edit projects, tasks, and tags to observe changes.
  2. On the Profiles page by deleting, exporting, or renaming profiles.

What was the expected result?

No response

Put here any screenshots or videos (optional)

IMG-20240217-WA0002 IMG-20240217-WA0003 IMG-20240217-WA0004 IMG-20240217-WA0005 IMG-20240217-WA0006 IMG-20240217-WA0007 IMG-20240217-WA0008 IMG-20240217-WA0009

How can we contact you (optional)

No response

Would you like to work on this issue?

Yes

By submitting this issue, I have confirmed that:

  • I have starred the repo ⭐ and watched 👀 it on GitHub and followed the contribution guidelines.

@Pavel401 can I work on this issue?

Currently, the app's color, themes, and fonts are hardcoded, making it difficult to manage the overall theme. It would be helpful if we could create system-wide global classes to easily handle these elements. For instance, we could create a class called "TaskWarriorColors" and list all the static color values under it. This way, we could easily call the colors we need, such as TaskWarriorColors.black or TaskWarriorColors.white. The same approach could be used for fonts as well.

@its-me-abhishek you can make multiple prs and issues for that.

I think we can use extension method on alert dialog widget and create all UI designs like,be it colour or be it border radius and other elements I think it will be the most optimised code
@Pavel401 what do you think?

I think we can use extension method on alert dialog widget and create all UI designs like,be it colour or be it border radius and other elements I think it will be the most optimised code
@Pavel401 what do you think?

Discuss with @its-me-abhishek about the most effective implementation.

@its-me-abhishek instead of creating those border radius and colors in every dialog and other elements that are common,you can use extension method on alert dialog and easily use the same code again and again.what do you think and the most optimised one can be created a alert dialog util.dart and then pass the title text,call back functions and then import this in necessary files.Actually i wanted to work on this but helping other team mates can also be a great choice.what do you think @its-me-abhishek and @Pavel401

hey, major portions that are being repeated in most of the dialog boxes are the surfaceTintColor, shadowColor, backgroundColor and some text styles, instead of making a function, I've updated the main.dart, to include a config for dialogTheme in the ThemeData, and have updated the values there.

Edit: You might make a function after the addition of FontStyles, as main.dart won't work without hot reload in Statless widgets so, we'll have to make a function anyways.