-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Changed: New repo description design #2879
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
Conversation
|
The screenshot seem to be from a very old app version, urls are outdated. It also seems unnecessary to include the android keyboard in it. Maybe a screenshot from a (landscape) tablet would have a form-factor more suited for most screens |
|
And please follow commit message style, see git history |
|
The images are way too big and taking lot of space. And https://github.com/termux/termux-app/blob/5e0749ac0ddd1cd5b93a2c11eb5e6f02d718075b/README.md |
|
Marked as draft until the decision of new motd design to avoid divergent screenshots. |
5e0749a to
c997210
Compare
b125a72 to
d8184af
Compare
Your both reviews were attended 👍 |
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as off-topic.
This comment was marked as off-topic.
|
Whoops, can't I go and post my dissatisfied comments? Is this your official face? |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
|
@gascs feel free to learn some manners and get back with constructive feedback rather than just insults |
|
I think its worth noting that some people would prefer the classic motd while this new implementation is something new that old termuxistas wouldn't get used to. We've seen some reports regarding this. perhaps we could poll this out? while this change is pretty quite aestethic for some. I think the plain one is suited for me, secondly is zoom issues which this breaks the motd readability if there's not enough columns to print. |
|
@WMCBTech Check termux/termux-packages#11250 (comment) |
21804bc to
3f7a939
Compare
5a4d129 to
4c01bc8
Compare
Done some changes to the PR.
|
3668c04 to
64857ff
Compare
64857ff to
aef4f4b
Compare
aef4f4b to
e962d9c
Compare
Added some better badges to the description, center alignment using html, a screenshot from the app and a termux logo.
The logo uses a Open Source font called Lexend.