flutter_settings_screens icon indicating copy to clipboard operation
flutter_settings_screens copied to clipboard

notification bugfix and colors parameter

Open ToH2002 opened this issue 3 years ago • 2 comments

Hi there, I created a branch in order to fix a notification bug: notifications didn't affect tiles with a key that contained upper-case letters due to a "ToLower" in the function that lists the tiles to be notified. Since I generally use camelCase to name keys, this created issues.

Also, I added a parameter "colors" to ColorPickerSettingsTile, so users can choose their own color selection.

All other changes are just automatic formatting made by VSCode - sorry about that...

Hope you can incorporate this!

ToH2002 avatar Nov 22 '22 14:11 ToH2002

@ToH2002 Hi, I tried to review the changes but the code submitted in PR is poorly formatted, can you please take a minute & check the formatting issue & re-submit?

Also, Take a latest pull before updating the PR as there seems to be some conflicts visible in the PR.

Thanks. 😄

GAM3RG33K avatar Sep 11 '24 06:09 GAM3RG33K

I'll work with the latest version and create a pull request once done

Cheers

Torsten

ToH2002 avatar Sep 11 '24 10:09 ToH2002