Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

editor: new wrapper toggle-switch #120

Merged
merged 1 commit into from
Jan 28, 2020

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Jan 22, 2020

  • Add a new 'toggle-switch' wrapper

Co-Authored-by: Renaud Michotte [email protected]

Why are you opening this PR?

  • Allow to hide/view a set of fields using a switch button not defined into json schema

How to test?

image

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@zannkukai zannkukai added the wip label Jan 22, 2020
@zannkukai zannkukai force-pushed the zan-#UCLpaging-new-features branch 2 times, most recently from ba7b96b to 1fbc49b Compare January 22, 2020 11:17
@zannkukai zannkukai requested review from Garfield-fr, jma and sebdeleze and removed request for jma January 22, 2020 11:17
@zannkukai zannkukai self-assigned this Jan 22, 2020
@zannkukai zannkukai requested a review from lauren-d January 22, 2020 11:18
@zannkukai zannkukai removed the wip label Jan 22, 2020
@Component({
selector: 'ng-core-editor-formly-toggle-wrapper',
template: `
<div class='toggle-wrapper'>
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not find the styles of toggle-wrapper

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No specific styles are used for toggle-switch.

If bootstrap toggle library is loaded for projec, the display will be a switch (like the screenshoot), otherwise a default checkbox will be displayed.

* Add a new 'toggle-switch' wrapper

Co-Authored-by: Renaud Michotte <[email protected]>
@zannkukai zannkukai force-pushed the zan-#UCLpaging-new-features branch from 1fbc49b to 870f169 Compare January 22, 2020 20:13
@zannkukai zannkukai requested a review from jma January 22, 2020 20:30
@sebdeleze sebdeleze merged commit 98c1d9f into rero:dev Jan 28, 2020
@zannkukai zannkukai deleted the zan-#UCLpaging-new-features branch February 14, 2020 08:26
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.

4 participants