Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Conversation

@mxstbr
Copy link
Contributor

@mxstbr mxstbr commented May 14, 2018

Status

  • WIP
  • Ready for review
  • Needs testing

TODO

  • Render mobile message replies
    • Collapse long quotes
    • Fix width of quoted message
  • Style chat input
  • Render currently quoted message in chat input
  • Send message with parent ID
  • Add action UI for replies to message bubbles

Current status:

@mxstbr mxstbr changed the title Mobile message replies Render mobile message replies May 16, 2018
ryota-murakami
ryota-murakami previously approved these changes May 17, 2018
${props => (props.expanded ? '' : 'max-height: 100px;')};
`;

export const QuoteGradient = styled(LinearGradient).attrs({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, i don't know Expo provided LinearGradient & there is such as cool way to handle CSS Gradients by styled-components on ReactNative😋

@mxstbr mxstbr mentioned this pull request May 22, 2018
5 tasks
@mxstbr mxstbr mentioned this pull request May 23, 2018
15 tasks
"dev": true,
"minify": false,
"urlRandomness": "66-qcw"
"urlRandomness": "2s-bjc"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this have notes around why this needs to be this specific thing?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's automatically generated by Expo

@brianlovin
Copy link
Contributor

Overall this looks good to me @mxstbr - I think you'll have the fastest ability to resolve those conflicts, then we can get this merged :)

@mxstbr
Copy link
Contributor Author

mxstbr commented May 23, 2018

Will resolve and merge tomorrow!

brianlovin
brianlovin previously approved these changes May 23, 2018
@mxstbr mxstbr merged commit a114994 into alpha May 24, 2018
@mxstbr mxstbr deleted the mobile-message-replies branch May 24, 2018 07:12
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.

4 participants