Skip to content
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

[Wallet] Fix top of emojis cut off in the activity feed #1243

Merged
merged 2 commits into from
Oct 8, 2019

Conversation

jeanregisser
Copy link
Contributor

Description

This fixes the top of emojis being cut off in the activity feed.

This was related to the problem we have with texts using our custom font when lineHeight is set.
I've opened #1242 to track it.

Tested

before
iTerm2 jBCgLg

after
iTerm2 YUCw10

Other changes

None

Related issues

Backwards compatibility

Yes.

@codecov
Copy link

codecov bot commented Oct 8, 2019

Codecov Report

Merging #1243 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1243   +/-   ##
=======================================
  Coverage   66.47%   66.47%           
=======================================
  Files         262      262           
  Lines        7648     7648           
  Branches      509      509           
=======================================
  Hits         5084     5084           
  Misses       2462     2462           
  Partials      102      102
Flag Coverage Δ
#mobile 66.47% <100%> (ø) ⬆️
Impacted Files Coverage Δ
...kages/mobile/src/transactions/TransferFeedItem.tsx 83.63% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff03623...87c6f1e. Read the comment docs.

@jeanregisser jeanregisser merged commit 222ab26 into master Oct 8, 2019
@jeanregisser jeanregisser deleted the jeanregisser/fix-emoji-cut-off branch October 8, 2019 13:47
aaronmgdr added a commit that referenced this pull request Oct 8, 2019
* master: (35 commits)
  [Wallet] Fix top of emojis cut off in the activity feed (#1243)
  Adding a contract to store minimum required client version (#1081)
  Revert "Feature #909 proxy delegatecall (#1152)" (#1241)
  Use ContractKit to get addresses for Blockchain API (#1175)
  Feature #909 proxy delegatecall (#1152)
  Fix Faucet done message (#1217)
  Updated SETUP.md with new yarn process (#1224)
  Adding `increaseAllowance` and `decreaseAllowance` methods (#1196)
  extracting function signatures (#1061)
  Fix integration hardcode (#1208)
  Fixing flaky governance test (#1155)
  Restore CI branch (#1223)
  [wallet] e2e back to green (#1210)
  [Wallet] Implement new import wallet flow designs (#1209)
  [Wallet] Fix disable conditions for butons on Enter Invite screen (#1214)
  [protocol] Rename infrastructureFraction to proposerFraction (#1174)
  [ck] Proper promise treatment to avoid UnhandledPromises (#1219)
  [ck] Transform StableToken parameters from fixidity format (#1218)
  [wallet]Store encrypted local signing key (#1188)
  2019-10-03 alfajores deployment (#1200)
  ...
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.

Top of emojis cut off in activity feed
2 participants