search

BlackyHawky / Clock

Privacy-conscious open-source clock, based on AOSP Clock
Apache License 2.0
178 stars 11 forks source link

Smaller font in alarm list #91

Closed arnoulfos closed 3 weeks ago

arnoulfos commented 1 month ago

Hi , I would like a smaller font in the alarm list page to see all alarms without scrolling when I have a lot thanks in advance

BlackyHawky commented 1 month ago

What text(s) are you talking about?

I won't change the height of the elements that can be touched because they respect Android's recommendations, i.e. 48dp (see here)

arnoulfos commented 1 month ago

All right but at least the elements could be organized in a more compact way that fit more alarms in the list visible in the screen. Thanks in advance

BlackyHawky commented 1 month ago

I'm sorry, but I don't see how I can rearrange the alarms. There's already a reduced view showing the main information and an extended view to show more options.

If I reduce anything, someone else will most likely open a problem saying it's too small. I hope you understand that I can't please everyone with all the different screen sizes.

arnoulfos commented 1 month ago

I have a very big screen but on my screen in the list fit 4 alarms only with the app while 6 in the stock app (screenshot)

BlackyHawky commented 1 month ago

I'll give it a try in a few days, then offer you the opportunity to test a beta version and let me know what you think.

arnoulfos commented 1 month ago

Thanks a lot!

Aug 26, 2024 12:35:09 BlackyHawky @.***>:

I'll give it a try in a few days, then offer you the opportunity to test a beta version and let me know what you think.

— Reply to this email directly, view it on GitHub[https://github.com/BlackyHawky/Clock/issues/91#issuecomment-2309778073], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ARJC4UW44BX4W4XHDFCZF4LZTLZEZAVCNFSM6AAAAABNCQU7QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBZG43TQMBXGM]. You are receiving this because you authored the thread. [Tracking image][https://github.com/notifications/beacon/ARJC4UQJAZBXDF3PFHAKIV3ZTLZEZA5CNFSM6AAAAABNCQU7QWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJVRVJS.gif]

BlackyHawky commented 1 month ago

@arnoulfos Can you test the attached apk and give me your opinion please?

You should now have 5 alarms displayed.

To get 6, I'd have to play with the height of the title, but I won't do that because I don't want to change the minimum height of the elements that can be touched as indicated in my previous comment.

In any case, you should find the display much better now.

Clock_2.6-debug.zip

arnoulfos commented 1 month ago

Yes it is a bit better , thanks a lot. And could not be reduced a little bit the white space between the alarms?

Aug 29, 2024 01:56:43 BlackyHawky @.***>:

@arnoulfos[https://github.com/arnoulfos] Can you test the attached apk and give me your opinion please?

You should now have 5 alarms displayed.

To get 6, I'd have to play with the height of the title, but I won't do that because I don't want to change the minimum height of the elements that can be touched as indicated in my previous comment.

In any case, you should find the display much better now.

Clock_2.6-debug.zip[https://github.com/user-attachments/files/16789798/Clock_2.6-debug.zip]

— Reply to this email directly, view it on GitHub[https://github.com/BlackyHawky/Clock/issues/91#issuecomment-2316376582], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ARJC4UTXFYKT7KTOIPJUTCDZTZISTAVCNFSM6AAAAABNCQU7QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGM3TMNJYGI]. You are receiving this because you were mentioned. [Tracking image][https://github.com/notifications/beacon/ARJC4UXJSQV7KBWUMOQVGALZTZISTA5CNFSM6AAAAABNCQU7QWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUKCENAM.gif]

BlackyHawky commented 1 month ago

Currently, the alarm list is separated by a space equal to 10dp (as are the city and timer lists).

Maybe I can reduce this value to 8dp but I won't never go below 8dp. I'll have to test again and keep you informed.

arnoulfos commented 1 month ago

All right, thanks a lot!

Aug 29, 2024 11:06:02 BlackyHawky @.***>:

Currently, the alarm list is separated by a space equal to 10dp (as are the city and timer lists).

Maybe I can reduce this value to 8dp but I won't never go below 8dp. I'll have to test again and keep you informed.

— Reply to this email directly, view it on GitHub[https://github.com/BlackyHawky/Clock/issues/91#issuecomment-2316968394], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ARJC4USDYQ6CFOT5VM7JTVTZT3I6TAVCNFSM6AAAAABNCQU7QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWHE3DQMZZGQ]. You are receiving this because you were mentioned. [Tracking image][https://github.com/notifications/beacon/ARJC4UWHBWY547YVX4GYS3TZT3I6TA5CNFSM6AAAAABNCQU7QWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUKDIQ4U.gif]

BlackyHawky commented 3 weeks ago

@arnoulfos You can test the attached file again; all the margins of each view have been reduced to 8dp. I won't go below this value.

Clock_2.6-debug.zip

arnoulfos commented 3 weeks ago

Yes thanks a lot , just the top line that writes  alarm at the top is unuseful, the "alarm" text and icon  could be in bold or another color to show that we are on this tab and the "setting " icon could be put at the bottom instead of the "bedtime" text and icon , as you know the proverb  : "less is more"! Thanks a lot

Sep 7, 2024 16:56:35 BlackyHawky @.***>:

@arnoulfos[https://github.com/arnoulfos] You can test the attached file again; all the margins of each view have been reduced to 8dp. I won't go below this value.

Clock_2.6-debug.zip[https://github.com/user-attachments/files/16918506/Clock_2.6-debug.zip]

— Reply to this email directly, view it on GitHub[https://github.com/BlackyHawky/Clock/issues/91#issuecomment-2335195070], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ARJC4UR4USCAX6UTQIXHV73ZVMAZDAVCNFSM6AAAAABNCQU7QWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZVGE4TKMBXGA]. You are receiving this because you were mentioned. [Tracking image][https://github.com/notifications/beacon/ARJC4URL77GZ7T3YBNYQHX3ZVMAZDA5CNFSM6AAAAABNCQU7QWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTULGA734.gif]

BlackyHawky commented 3 weeks ago

Yes thanks a lot , just the top line that writes alarm at the top is unuseful, the "alarm" text and icon could be in bold or another color to show that we are on this tab and the "setting " icon could be put at the bottom instead of the "bedtime" text and icon , as you know the proverb : "less is more"! Thanks a lot

Your comment is not related to your original request. We mustn't mix topics.

In any case, I won't be changing the current main interface.