Skip to content

chore: Address comments in #3911#7690

Merged
tofumatt merged 2 commits intomasterfrom
yoavf-update/blocks-alignment-rtl
Jul 3, 2018
Merged

chore: Address comments in #3911#7690
tofumatt merged 2 commits intomasterfrom
yoavf-update/blocks-alignment-rtl

Conversation

@tofumatt
Copy link
Copy Markdown
Member

@tofumatt tofumatt commented Jul 3, 2018

This is just #3911 but with comments fixed. #3911 is approved but I couldn't push comments tweaks to it so it's here. 🤷‍♂️

Description

I merged #3911 but these are comment tweaks I requested in that PR.

@tofumatt tofumatt added this to the 3.2 milestone Jul 3, 2018
@tofumatt tofumatt requested review from a team and removed request for a team July 3, 2018 13:57
@tofumatt
Copy link
Copy Markdown
Member Author

tofumatt commented Jul 3, 2018

All that work to do this and then I merged the old PR absentmindedly. sigh 😔

@tofumatt tofumatt force-pushed the yoavf-update/blocks-alignment-rtl branch from f4d1e44 to 2d59a39 Compare July 3, 2018 14:21
@tofumatt tofumatt force-pushed the yoavf-update/blocks-alignment-rtl branch from 2d59a39 to fa59f48 Compare July 3, 2018 14:22
@tofumatt tofumatt changed the title RTL: Fix left/right alignments of blocks chore: Address comments in #3911 Jul 3, 2018
@tofumatt tofumatt merged commit a713494 into master Jul 3, 2018
@tofumatt tofumatt deleted the yoavf-update/blocks-alignment-rtl branch July 3, 2018 16:19
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.

2 participants