-
Notifications
You must be signed in to change notification settings - Fork 112
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
new default icons for selected default labels + filter counter for overview in drawer #182
base: develop
Are you sure you want to change the base?
Conversation
…lytics (CCExtractor#169) * test: Added tests for api service * feat: Added Firebase analytics and crashlytics * feat: Added environment variables for sensitive info * Create .env
* test: Added tests for api service * feat: Added Firebase analytics and crashlytics * feat: Added environment variables for sensitive info * Create .env * Delete .env * release: v0.9.1 beta release
@AbdulMalikDev added the PR here |
@Rushour0 please resolve the conflicts. |
Done @AbdulMalikDev |
@Rushour0 Please also make sure the PR is passing all checks. |
Alright my bad, I will check it again, it was working some time ago |
@AbdulMalikDev cleared up the flutter analyze error |
@AbdulMalikDev could you let me know the status of this feature addition |
Added default label icons
Made a new enum type - DefaultLabels which maps to the respective png files
A new default label can be added in DefaultLabels, with its corresponding png file in assets/icons with the same name as the label
Example :
label: videos
filename: videos.png
Added a class DefaultPaths
The intelli-sense can make it faster to write these paths, than having to remember them.
Example:
normal string path: assets/icons/
with DefaultPaths: DefaultPaths.defaultLabelIcons
The variable can be autocompleted with intellisense and reduces the need to remember the path if the variables are well named
solution to issue #108
New function addition in the torrent service to count the torrents by filter
All counts of the filters are now visible in the drawer
issue #109 solution