Skip to content

Decidim shorten Last Activity Cards's description#5742

Merged
tramuntanal merged 9 commits intodecidim:masterfrom
CodiTramuntana:5672_decidim_shorten_activity_cards
Feb 19, 2020
Merged

Decidim shorten Last Activity Cards's description#5742
tramuntanal merged 9 commits intodecidim:masterfrom
CodiTramuntana:5672_decidim_shorten_activity_cards

Conversation

@Leusev
Copy link
Copy Markdown
Contributor

@Leusev Leusev commented Feb 17, 2020

🎩 What? Why?

Shorten 100 default's last activity cards length description to 80 chars, in order to reduce default description to two lines.
As asked in #5672, only focusing on Desktop view.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

@Leusev Leusev marked this pull request as ready for review February 17, 2020 10:10
@Leusev Leusev mentioned this pull request Feb 17, 2020
5 tasks
@Leusev Leusev requested a review from tramuntanal February 17, 2020 11:42
tramuntanal
tramuntanal previously approved these changes Feb 18, 2020
@tramuntanal
Copy link
Copy Markdown
Contributor

tramuntanal commented Feb 18, 2020

@Leusev merge master in order to get the tests succeed please

Also, there was no test regarding this already existing truncate?

@tramuntanal tramuntanal merged commit 905be21 into decidim:master Feb 19, 2020
@tramuntanal tramuntanal deleted the 5672_decidim_shorten_activity_cards branch February 19, 2020 09:13
microstudi pushed a commit to Platoniq/decidim that referenced this pull request Feb 21, 2020
* [FIX] shorten 100 chars description max lenght in last activity cards to 80 chars max limit

* [FIX] shorten 100 chars description max lenght in last activity cards to 80 chars max limit

* [FIX] Wrong push, undone my last commit changes - My fault, sorry.

* [FEAT] Shorten the 100 chars default last activity cards description length to 80 chars

* [FIX] Fixed changelog issue reference

* [FIX] fixed CHANGELOG desc

Co-authored-by: ramoncodit <37654424+ramoncodit@users.noreply.github.com>
@andreslucena andreslucena mentioned this pull request Aug 4, 2020
13 tasks
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.

3 participants