falzonv / discreet-launcher

Enjoy a clean home screen while accessing everything in an instant! (Profitez d'un écran d'accueil épuré tout en accédant à tout en un instant !)
https://vincent-falzon.com
GNU General Public License v3.0
218 stars 41 forks source link

Automatically adjust the number of columns to screen size #67

Closed xi7 closed 3 years ago

xi7 commented 3 years ago

Screenshot_2021-06-09-07-28-51

https://user-images.githubusercontent.com/67331785/121337697-3105bd80-c8f3-11eb-99a1-1db7f0500976.mp4

row landscape = 7 or 8 and not 5 (and also folder with fit windows)

or possibility change by settings

xi7 commented 3 years ago

and when possible test the home favorites only with icons but fit layout size.

https://user-images.githubusercontent.com/67331785/121393444-b8b8ef80-c926-11eb-836d-651e4419a954.mp4 always consider better screen use

xi7 commented 3 years ago

Screenshot_2021-06-09-08-48-24 portrait mode - home favorites waste of layout, half of screen with almost nothing this is a favorites selected manually, i think no need be a mirror of grid_view with only 4 row and names and small icon.

xi7 commented 3 years ago

Screenshot_2021-06-10-07-34-09

portrait w/ 7 and no names.

falzonv commented 3 years ago

Hello,

Ok, thank you for the screenshots, videos and explanations :-)

The problem with a fixed number of columns is that it would not work well in all cases, for example 7 would be too much for some small screens, and I agree that 5 is too low on bigger screen.

I will try to have the number of columns automatically adjusted to the screen width (and updated when the user rotates the screen), and also updated when the option to hide app names is enabled/disabled.

This will probably be pat of the next release.


For now I like the current favorites panel and do not plan to change it. Indeed it may take a bit of space depending on the number of icons, but it is hidden by default which leaves the home screen completely empty.

Also, the notification popup already provides a more condensed view of the favorites. To be honest this was not intended but, because it is displayed over other applications, I have very few control over the appearance of the notification popup (I cannot even set it full width to match the usual folders: Android automatically adds margins around the popup when it is displayed).

Best regards.

xi7 commented 3 years ago

my phone have 4 inch, work very well with 10x landscape and 6x portrait, agradable visualization and usable while no names.

falzonv commented 3 years ago

Yes, when the names are hidden there can be more applications, but 6x portrait with the names doesn't look very good even on my 5.2" phone... This is why I will try to make it dynamic, so that the limit is updated automatically when the user enable/disable apps names.

xi7 commented 3 years ago

the columns to screen size is only while no names. with names fix only the landscape with 7x and portrait 4x.
if no work automatically enable function manually.

*possible fix the notification area size folder view ? use the same size grid_view folder.

xi7 commented 3 years ago

possible make 1 gesture and call the folder notification ?

falzonv commented 3 years ago

the columns to screen size is only while no names. with names fix only the landscape with 7x and portrait 4x. if no work automatically enable function manually.

I am bit surprised because for other things you were very pushy to have me remove/reduce code already optimized, and here I work on a solution which should cover all cases simply (portrait/landscape, with/without names) and you would prefer to have two different behaviors (with/without names), perhaps with an additional setting, etc. That is not very logical.

The first small tests are quite positive anyway, so I plan to do a common solution based on screen size.

In addition, I am thinking about adding an option "Remove margins". Personally I don't like the applications being too close like on your last screenshot (maybe I have big fingers :joy:), but you seem to prefer it (to each his own). => Would you like to have an option like this?


*possible fix the notification area size folder view ? use the same size grid_view folder.

For now I was not able to fix this unfortunately, this is also something I am not really happy with :-/ This is what I explained in a previous comment:

because it is displayed over other applications, I have very few control over the appearance of the notification popup (I cannot even set it full width to match the usual folders: Android automatically adds margins around the popup when it is displayed).

I have made many tries but for now there were only two results:

The best would be a mix of the two, but for now I was not able to do it (tried a couple of hours, but there are also many other requests so I decided to let it like this for now).


possible make 1 gesture and call the folder notification ?

From the home screen: yes, this is already existing with the favorites panel. From another application: no, I am not able to do this sorry.

falzonv commented 3 years ago

After further tests, I think the result will be something like this:

At the end, it is not very far from what you suggested :-)

xi7 commented 3 years ago

I am bit surprised because for other things you were very pushy to have me remove/reduce code already optimized, and here I work on a solution which should cover all cases simply (portrait/landscape, with/without names) and you would prefer to have two different behaviors (with/without names), perhaps with an additional setting, etc. That is not very logical.

i just change smali file 0x4 to 0x6 portrait and 0x5 to 0x7 landscape. (no names situation)

but i prefer landscape with x10 , tablet and phone

xi7 commented 3 years ago

have print notification test ?

xi7 commented 3 years ago

https://user-images.githubusercontent.com/67331785/121743632-058ff800-cad8-11eb-8ad6-36a4c6005d84.mp4

that my problem with notification area, very small cant scroll icons

xi7 commented 3 years ago

https://user-images.githubusercontent.com/67331785/121771364-50942480-cb45-11eb-8609-edba6f7a2f36.mp4 landscape 5x9 icons 60dp

https://user-images.githubusercontent.com/67331785/121771394-7de0d280-cb45-11eb-9820-32068e6019c6.mp4 portrait 9x5 icons 60dp


**#67 no names good solution, big and organized

about with names i dont have solution because the system interferes with the font size**

xi7 commented 3 years ago

Screenshot_2021-06-13-09-57-32

manually full solution

falzonv commented 3 years ago

possible fix the notification area size folder view ? use the same size grid_view folder.

=> Good news, after comparing dozens of parameters in Android styles and many more tests, the notification width is now fixed! It seems to work well on all versions, but on my phone I had to perform a reboot before it was ok, strange... Anyway, the issue with the width will be solved in the next release :-)

that my problem with notification area, very small cant scroll icons

=> Ok, I will set a minimum height to the notification popup and it should solve this issue.

\ Regarding the last screenshot, sorry but I do not plan to implement all these options (except hide apps name which is already done, and I will add an option to remove or not margins when apps names are hidden). Most users do not need such detail of customization (text size, icon size, ...) to be happy with the launcher and it would add a lot of complexity (and possible bugs) in the code for future maintenance.

xi7 commented 3 years ago

From the home screen: yes, this is already existing with the favorites panel. From another application: no, I am not able to do this sorry.

is not possible the button home open the notification folder without permission when out home ?

xi7 commented 3 years ago

sorry i clicked wrong and close, how to back ?

falzonv commented 3 years ago

is not possible the button home open the notification folder without permission when out home ?

No sorry, this is not possible to override the home button. I think this is to prevent malicious apps from blocking the user phone (if it overrides the home button, the back button and maybe some others, the user will not be able to leave the malicious app without rebooting the phone ; and if the app has permission to run on startup, then the user is really locked).


Regarding other things from this ticket, I am pleased to inform you that this has been implemented in new release v4.1.0! This release has been published a few minutes ago.

Here are the usual availability delays for each location:

Remarks: I will leave this ticket opened until the release is effectively published on F-Droid. You can use the export/import feature to switch from one version to another.

Best regards.

xi7 commented 3 years ago

4.1.0 perfect !

xi7 commented 3 years ago

Screenshot_2021-06-19-07-42-58

android 5 notification area i have target black folder

falzonv commented 3 years ago

Thank you for your feedback! :-)

android 5 notification area i have target black folder

Strange, I was not yet able to reproduce it with the Android 5 emulator :-/ (I have a small shadow added by Android above and below the notification area, but not black rectangles like on your screenshot)

Do you also observe this bug when opening a folder from the notification area?

xi7 commented 3 years ago

https://user-images.githubusercontent.com/67331785/122644385-4ea10700-d0eb-11eb-9971-5576030f1697.mp4

here show only black target

xi7 commented 3 years ago

Screenshot_2021-06-19-11-26-17

home favorites work perfect but with notifications area black and a big white text area while landscape

falzonv commented 3 years ago

android 5 notification area i have target black folder

=> I tried something to fix this in v4.1.1 if you want to check. On my side it removed the shadow, so I hope it will remove the black for you. If not, I have no other idea for now :-/

big white text area while landscape

=> For me this happens only when clicking the search icon in landscape mode, and only with some keyboards (for example, it does it with the standard Google keyboard but not with Hacker's Keyboard). This is not a bug I will be able to fix for the search feature, as it is not on Discreet Launcher side but on keyboard side.

The keyboard should be shown only after you clicked the search icon, not immediately when opening the notification.

xi7 commented 3 years ago

app package problem, i cant install 4.1.1, no open

falzonv commented 3 years ago

Ok, thank you for the report! After regenerating the APK, the size is slightly different so probably something was missing (but no idea what, I clicked the same generation button both times and there was no changes in between...).

Anyway, I replaced the APK file from the Releases page and now it should work normally.

xi7 commented 3 years ago

problem solved now. working.

xi7 commented 3 years ago

=> For me this happens only when clicking the search icon in landscape mode, and only with some keyboards (for example, it does it with the standard Google keyboard but not with Hacker's Keyboard). This is not a bug I will be able to fix for the search feature, as it is not on Discreet Launcher side but on keyboard side.

The keyboard should be shown only after you clicked the search icon, not immediately when opening the notification.

after test with a lot keyboard, this is a layout problem you need fix it. (landscape mode) the portrait work fine.

xi7 commented 3 years ago

https://user-images.githubusercontent.com/67331785/122759654-9309df80-d270-11eb-87c5-cde115dcc83f.mp4 landscape bad layout

https://user-images.githubusercontent.com/67331785/122759692-9bfab100-d270-11eb-80f3-9850bce59c53.mp4 portrait

falzonv commented 3 years ago

Ok, thank you for the tests and feedback. I have found a workaround that should work with most keyboards, it will be implemented in the next release.

xi7 commented 3 years ago

maybe always portrait search solve. my test with Hacker's Keyboard the problem still persistent

xi7 commented 3 years ago

its possible choose manually what to do the swipe up/down/ and notification ? -direct search -show home favorites -show grid view -and notification... favorites or direct search -none

falzonv commented 3 years ago

Sorry, the up/down navigation is not something that I plan to change.

If I do ticket #73, it would add possibility to launch one application with "swipe left" or "swipe right", so the user could assign the "Search" application to one of these swipes and have direct search from the home screen in only one swipe.

Regarding the notification opening favorites or direct search, I will have to check. If this can be done without too much duplicate code I am ok to implement this option. => Created ticket #75 for follow-up.


Since Thursday, release v4.1.1 is available on F-Droid so I will close this ticket. Feel free to add comments if needed or wished.