Skip to content

ci(snap): fix version and update README#932

Merged
coder3101 merged 6 commits intomasterfrom
fix-snap-ver
Jul 15, 2021
Merged

ci(snap): fix version and update README#932
coder3101 merged 6 commits intomasterfrom
fix-snap-ver

Conversation

@coder3101
Copy link
Copy Markdown
Member

No description provided.

@coder3101 coder3101 linked an issue Jul 15, 2021 that may be closed by this pull request
4 tasks
@coder3101 coder3101 changed the title ci(snap): Fix version and update README ci(snap): fix version and update README Jul 15, 2021
@coder3101 coder3101 enabled auto-merge (squash) July 15, 2021 03:55
@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

The "Get it from" is invisible with a height of 28px. Do we need that line?

@coder3101
Copy link
Copy Markdown
Member Author

coder3101 commented Jul 15, 2021

The "Get it from" is invisible with a height of 28px. Do we need that line?

Not necessary but why do we keep it along with downloads, sponsors and other badges. I suggest we put it in a new line below so it is visible completely and centered to draw focus of visitors.

@coder3101
Copy link
Copy Markdown
Member Author

Something like this:
Screenshot from 2021-07-15 09-51-07

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

Yeah, but I'm wondering why we should draw attention to it. The badge somehow sounds like "you need to get it from snap store", but actually the snap installation method is targeted at only a few users.

@coder3101
Copy link
Copy Markdown
Member Author

We set the width=15% and use the second row in README for listing badges with store presence.
Screenshot from 2021-07-15 10-42-45

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

I think the badges in the version table are enough.

@coder3101
Copy link
Copy Markdown
Member Author

coder3101 commented Jul 15, 2021

I think the badges in the version table are enough.

Fine, remove the "Get it from Snap store" badge, update language READMEs and approve.

@coder3101 coder3101 merged commit 089f355 into master Jul 15, 2021
@coder3101 coder3101 deleted the fix-snap-ver branch July 15, 2021 09:10
@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

I reverted the "fix version" part by mistake...

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

I reverted the "fix version" part by mistake...

However, it's a good example that we should do one thing per commit.

toto6038 pushed a commit to l10n-tw/cpeditor that referenced this pull request Jul 18, 2021
coder3101 added a commit that referenced this pull request Jul 21, 2021
* init README_zh-TW.md

* add DONATE_zh-TW

* add language link for Chinese Traditional (Taiwan)

* finish translating CONTRIBUTION.md

* add translation for conventional commit

* Add link for Chinese Simplified (China)

* Add link to Chinese Traditional

* Fix snap version and update README (#932)

* build(snap): fix version by adding git (#933)

* Add snap

* Add link for zh-TW in README and edit the link text to 正體中文

* Change the link to zh_tw version on cpeditor.org

* Add zh-TW in the "files" part in ".all-contributorsrc: file"

* Add zh-TW description in dist/linux/cpeditor.desktop and cmake/cpeditor.appdata.xml.in

Co-authored-by: Ashar <[email protected]>
Co-authored-by: Yufan You <[email protected]>
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.

Wrong snap version: Could NOT find Git (missing: GIT_EXECUTABLE)

3 participants