We add the styles for spoiler text on native by adding a style for both the text color and the background to make the spoilers uniquely
display.
https://linear.app/comm/issue/ENG-1984/add-spoiler-style-to-native-markdown-styles
Depends on D5347
Paths
| Differential D5373 Authored by rohan on Oct 14 2022, 7:52 PM.
Details Summary We add the styles for spoiler text on native by adding a style for both the text color and the background to make the spoilers uniquely https://linear.app/comm/issue/ENG-1984/add-spoiler-style-to-native-markdown-styles Depends on D5347 Test Plan
Diff Detail
Event TimelineHerald added subscribers: tomek, ashoat. · View Herald TranscriptOct 14 2022, 7:52 PM2022-10-14 19:52:17 (UTC-7) rohan edited the summary of this revision. (Show Details)Oct 14 2022, 7:53 PM2022-10-14 19:53:54 (UTC-7) Harbormaster returned this revision to the author for changes because remote builds failed.Oct 14 2022, 7:55 PM2022-10-14 19:55:31 (UTC-7) Harbormaster completed remote builds in B12813: Diff 17597.Oct 14 2022, 8:03 PM2022-10-14 20:03:01 (UTC-7) rohan edited the summary of this revision. (Show Details)Oct 14 2022, 8:03 PM2022-10-14 20:03:57 (UTC-7) Comment Actions Code looks good, but I think about changing the black to something like a dark gray. I think the black looks a bit harsh with the black background outside the message container. Defer to @atul for second take as well This revision now requires changes to proceed.Oct 15 2022, 10:51 AM2022-10-15 10:51:07 (UTC-7) Comment Actions
Good point I agree after taking another look at it Harbormaster completed remote builds in B12817: Diff 17601.Oct 15 2022, 4:59 PM2022-10-15 16:59:08 (UTC-7) Comment Actions Not sure if this is the "final" styling for spoilers, but looks good for now. I think we might want to round the corners and kind of match the styling of code blocks (eg same color) Comment Actions I think this is a fine style for now, we can revisit later
Comment Actions color looks a lot better, I agree with @ashoat and I think we can get rid of spoilerText style. I would address that before landing This revision is now accepted and ready to land.Oct 17 2022, 6:45 AM2022-10-17 06:45:38 (UTC-7)
Harbormaster completed remote builds in B12834: Diff 17619.Oct 17 2022, 8:45 AM2022-10-17 08:45:03 (UTC-7) rohan added a child revision: D5383: [native] Display the spoiler content on tap.Oct 17 2022, 12:31 PM2022-10-17 12:31:43 (UTC-7) Harbormaster completed remote builds in B13327: Diff 18278.Nov 9 2022, 10:19 AM2022-11-09 10:19:28 (UTC-8) Closed by commit rCOMM4a38e57103a1: [native] Add spoiler style to native markdown styles (authored by rohan). · Explain WhyNov 30 2022, 2:47 PM2022-11-30 14:47:22 (UTC-8) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 19045 native/markdown/rules.react.js
native/markdown/styles.js
native/themes/colors.js
|