Skip to content

[dev] PBI AB#20206 [My Ministry > Members] [mobile] - Convert table to new mobile table#57

Merged
oilywithraybans merged 2 commits intosaddlebackdev:devfrom
gbulatov:features/20206-page-content-scroll
Sep 12, 2019
Merged

[dev] PBI AB#20206 [My Ministry > Members] [mobile] - Convert table to new mobile table#57
oilywithraybans merged 2 commits intosaddlebackdev:devfrom
gbulatov:features/20206-page-content-scroll

Conversation

@IlyaRadinsky
Copy link
Copy Markdown
Contributor

@morethanfire Oops, I missed that the task https://saddlebackchurch.visualstudio.com/Church%20Management/_workitems/edit/20330 was removed 😕 so, if this feature not needed - just close this PR please.

Copy link
Copy Markdown

@oilywithraybans oilywithraybans left a comment

Choose a reason for hiding this comment

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

Dude, thanks for this work! A super simple feature, but I think it's a much needed one to facilitate us down the road. :shipit:

isFiltersRailOpen: PropTypes.bool,
scrollable: PropTypes.bool,
style: PropTypes.object,
};
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for adding the defaults and checks!

}
&.page--content-scrollable {
overflow-x: scroll;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I'm loving this new BEM naming convention! Event if .ui is sort of ruining it.

@oilywithraybans oilywithraybans merged commit 9285149 into saddlebackdev:dev Sep 12, 2019
@IlyaRadinsky IlyaRadinsky deleted the features/20206-page-content-scroll branch October 10, 2019 09:11
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