这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@TermuxMonet
Copy link

@TermuxMonet TermuxMonet commented Jul 11, 2022

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.

@TermuxMonet TermuxMonet changed the title New repo description New repo description design Jul 11, 2022
@Grimler91
Copy link
Member

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

@Grimler91
Copy link
Member

And please follow commit message style, see git history

@agnostic-apollo
Copy link
Member

The images are way too big and taking lot of space. And README is gonna get truncated when things are moved to docs. The UI pictures of different views would also be best suited in a dedicated UI docs page instead of README.

https://github.com/termux/termux-app/blob/5e0749ac0ddd1cd5b93a2c11eb5e6f02d718075b/README.md

@TermuxMonet TermuxMonet marked this pull request as draft July 12, 2022 11:57
@TermuxMonet TermuxMonet changed the title New repo description design Changed: New repo description design Jul 12, 2022
@TermuxMonet
Copy link
Author

Marked as draft until the decision of new motd design to avoid divergent screenshots.

@TermuxMonet TermuxMonet force-pushed the termux-description branch 3 times, most recently from b125a72 to d8184af Compare July 20, 2022 00:50
@TermuxMonet
Copy link
Author

TermuxMonet commented Jul 20, 2022

And please follow commit message style, see git history

Your both reviews were attended 👍

@TermuxMonet TermuxMonet marked this pull request as ready for review July 20, 2022 00:53
@gascs

This comment was marked as abuse.

@gascs

This comment was marked as abuse.

@agnostic-apollo

This comment was marked as off-topic.

@gascs
Copy link

gascs commented Aug 3, 2022

Whoops, can't I go and post my dissatisfied comments? Is this your official face?

@gascs

This comment was marked as spam.

@gascs

This comment was marked as spam.

@gascs

This comment was marked as spam.

@Grimler91
Copy link
Member

@gascs feel free to learn some manners and get back with constructive feedback rather than just insults

@ghost
Copy link

ghost commented Aug 3, 2022

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.

@agnostic-apollo
Copy link
Member

@WMCBTech Check termux/termux-packages#11250 (comment)

@TermuxMonet TermuxMonet force-pushed the termux-description branch 2 times, most recently from 5a4d129 to 4c01bc8 Compare February 13, 2023 14:04
@TermuxMonet
Copy link
Author

Done some changes to the PR.

@TermuxMonet TermuxMonet force-pushed the termux-description branch 2 times, most recently from 3668c04 to 64857ff Compare February 13, 2023 14:13
@Termux-Monet Termux-Monet closed this by deleting the head repository Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants