Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Limit card titles to two-lines with ellipsis/fade out #3241

Merged
merged 12 commits into from
Dec 5, 2018

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented Nov 24, 2018

Fixes #2340

Updates card titles to be a maximum of 2 lines high.

For webkit browsers (Chrome, Safari and also supported by Edge) we use the webkit multiline clamp CSS.

For firefox where this is not supported, we fade the end of the 2nd line out to indicate that it is truncated.

@nwmac nwmac self-assigned this Nov 24, 2018
@cfdreddbot
Copy link

✅ Hey nwmac! The commit authors and yourself have already signed the CLA.

@codecov
Copy link

codecov bot commented Nov 24, 2018

Codecov Report

Merging #3241 into v2-master will increase coverage by 0.01%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##           v2-master    #3241      +/-   ##
=============================================
+ Coverage      71.04%   71.05%   +0.01%     
=============================================
  Files            633      634       +1     
  Lines          27802    27816      +14     
  Branches        6330     6335       +5     
=============================================
+ Hits           19751    19765      +14     
  Misses          8051     8051

@richard-cox richard-cox merged commit b3b36a0 into v2-master Dec 5, 2018
@richard-cox richard-cox deleted the fix-text-label-overflow branch December 5, 2018 12:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants