Skip to content
This repository has been archived by the owner on Dec 12, 2022. It is now read-only.

BF Riot: Unable to open a file attachment of a room message #532

Merged
merged 3 commits into from
Mar 18, 2019

Conversation

SBiOSoftWhare
Copy link
Contributor

@SBiOSoftWhare SBiOSoftWhare requested a review from giomfo March 18, 2019 15:21
Copy link
Member

@giomfo giomfo left a comment

Choose a reason for hiding this comment

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

LGTM with one remark on a comment


if (tapPoint.y > 0 && tapPoint.y < self.messageTextView.frame.size.height)
// Use the same hack as `onMessageTap:`, check whether the current displayed text corresponds to an attached file
// NOTE: This assume that the cell has only one `MXKRoomBubbleComponent`
Copy link
Member

Choose a reason for hiding this comment

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

I would say: "This assumes that a cell with attachment has only one MXKRoomBubbleComponent"

@SBiOSoftWhare SBiOSoftWhare merged commit d3e2d6d into develop Mar 18, 2019
@SBiOSoftWhare SBiOSoftWhare deleted the riot_2338 branch March 18, 2019 16:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants