-
Notifications
You must be signed in to change notification settings - Fork 124
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
A better way/method can be implemented to display the applied filter info. #115
Comments
please assign this issue to me. |
Sure @sainideepanshu199 I am assigning you this issue. You can start by proposing the new suggested design and once the design gets approved then you can start coding/implementing it. |
@sainideepanshu199 there is some issue with qBittorent and flood which needs to be addressed and resolved until then you can use transmission instead of qBittorent. The setup is almost similar. |
@sainideepanshu199 is there any progress on this issue? Also @sainideepanshu199 were you able to solve the issue you were facing while setting up the project? If not then this is what you need to do to resolve the issue. Uninstall the older version of flood u have by using the command |
You have to enter a details that you use to create flood web-ui account. If it give this login error then try to uninstall flood using |
Unassigning this issue to @sainideepanshu199 as there was no progress reported on the issue |
Can you assign me this issue. |
Hello @Ronakkkkk please let me know if you still want to work on this issue - if yes then I will assign this issue to you. |
Please assign this issue to me. |
Please let me know what is your idea and how you are gonna improve it before I assign this issue to you. |
Hey @tb-rules10, thanks for proposing this but the reason why this was not followed and approved initially itself is that we want to save as much space as we can on the main screen to show torrents. This is the reason why search bar, download speed etc are hidden and appears on top swipe. Please suggest something that aligns with our minimal screen space usage goal. |
Hi @Akshatji800 its Aadarsh from FOSS 24. I think we can use chips to better portray the filtering options instead of the current radio selector. This proposal is also in line with using minimum space possible, since a user would have to scroll less. Below is a skeleton of what the filter would look like, as the tracker list would be varying in size, the filter dialog will be scroll-able and will use wrap widget to handle devices of varying width. |
Hey @Nailsonseat assigning this issue to you. But this issue is more about how we are gonna show the current filter selected in a better way. Currently it is shown in searchbar. I want you to work on that first. |
Thank you Akshat, I will come back to you with an idea regarding this very soon. |
@Nailsonseat good observation but don't you think that removing filter icon and putting a dismiss button instead of it creates confusion that what actually complete is? The idea is good that there should be a way to dismiss the filter (it is definitely increasing the UX) but you need to think of a better UI. |
Good point, the user will not know if 'complete' is a filter or something else, will work on it. I deleted the original post because I wanted to show a good way to show sorting too along with filters. I will come to you will a new idea for both soon |
@Akshatji800 I have some new mock-ups or rather old ones because these designs are more closer to the original UI.
FiltersI have two points with respect to the concern with confusion with dismiss button
Sort
Do let me know if more improvements can be made here |
i found a potential implementation for the sort button straight from the material 3 website. Just imagine this with icons, and without the text in the button. The focus is the button not the drop down itself demo.mp4I ll be on the lookout for better options |
Is your feature request related to a problem? Please describe.
The current way of displaying the applied filter info in the search bar can be improved/changed to make it look better.
Describe the solution you'd like
To be proposed by the contributors who want to take up the issue by commenting on the issue.
The text was updated successfully, but these errors were encountered: