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

[PRODDEV-150] Update filter breakpoints according to theme. #54

Merged
merged 3 commits into from
Apr 12, 2021

Conversation

manachynskyi
Copy link
Contributor

@manachynskyi manachynskyi commented Apr 1, 2021

Link to the task: https://openy.atlassian.net/browse/PRODDEV-150

Steps for review:

Copy link
Contributor

@duozersk duozersk left a comment

Choose a reason for hiding this comment

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

Great job! Please address the comments and we are good to go!

src/Plugin/Block/ActivityFinder4Block.php Outdated Show resolved Hide resolved
openy_af4_vue_app/src/components/ResultsBar.vue Outdated Show resolved Hide resolved
src/Plugin/Block/ActivityFinder4Block.php Outdated Show resolved Hide resolved
src/Plugin/Block/ActivityFinder4Block.php Outdated Show resolved Hide resolved
openy_af4_vue_app/src/App.vue Outdated Show resolved Hide resolved
openy_af4_vue_app/src/App.vue Show resolved Hide resolved
openy_af4_vue_app/src/components/Results.vue Outdated Show resolved Hide resolved
openy_af4_vue_app/src/components/Results.vue Outdated Show resolved Hide resolved
@@ -25,6 +26,12 @@ class ActivityFinder4Block extends BlockBase implements ContainerFactoryPluginIn

use EntityBrowserFormTrait;

const THEME_BOOTSTRAP_VERSION = [
'openy_carnation' => 4,
Copy link
Contributor

Choose a reason for hiding this comment

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

nice
But let's put this in config, please

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@podarok podarok merged commit bf8fe92 into ymcatwincities:4.x Apr 12, 2021
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.

None yet

3 participants