Skip to content

Conversation

@bertysentry
Copy link
Collaborator

  • Button size classes (btn-lg, btn-sm, etc.) are properly set to all elements as attribute size is specified
  • Toggle width and height automatically adjusts with the content size and font size (incl. a refresh once all fonts from the document are loaded)
  • Removed artifical min-height values for the various button sizes (it's automatic now)
  • Added support for empty buttons
  • Removed the font-family preset, as the toggle now properly adjust to any font type and size

@bertysentry
Copy link
Collaborator Author

Hi @ziscloud !
If you have time, could you please review this PR? It solves an issue with height that has been reported by several users (and reproduced by myself).

People were also asking if we could officially release this new version. How do we do this? (note: in my own dev environment, I set version to 0.3.0, so I wouldn't "mind" if you used the same version number... ;-) )

Thanks!

…n behavior to version 0.1.1

* Re-implemented the `style` attribute, which is now replaced with `toggle-class` (more logical and consistent with `on-class` and `off-class` (`style` is still supported for backward compatibility)
* Updated **index.html** to reflect the latest changes in the attributes
* Moved **gh-pages** branch content to **master/** with some updates and so so that index.html automatically uses /dist files
* Button size classes (`btn-lg`, `btn-sm`, etc.) are properly set to all elements as attribute `size` is specified
* Button width and height automatically adjusts with the content size and font size (incl. a refresh once all fonts from the document are loaded)
* Removed artifical `min-height` values for the various button sizes (it's automatic now)
* Added support for empty buttons
@bertysentry bertysentry force-pushed the feature/issue-5-height-does-not-work branch from 49f1230 to 360cb78 Compare October 2, 2018 16:46
@bertysentry
Copy link
Collaborator Author

Hi @ziscloud,

While waiting for your feedback, I made a few more changes:

  • I moved the GitHub pages to the main branch (for now, it's in the branch that I'm requesting you to merge). Now, index.html will use directly the compiled version of angular-boostrap-toggle in the /dist directory. This will make sure the GitHub pages always use the very latest version.
  • Doing so, I noticed a few minor and major issues that were introduced after the 0.1.1 version and which broke angular-bootstrap-toggle in several ways. Everything is fixed now and the index.html page now shows all controls as expected.

When you merge this PR to master, you will need to change the settings of the GitHub project, so that the GitHub pages don't refer to the gh-pages branch, but rather to master.

Thank you! I'm happy I could fix this great AngularJS directive and make it work as you originally intended! :-)

@ziscloud ziscloud merged commit 09ac150 into ziscloud:master Oct 8, 2018
@bertysentry bertysentry deleted the feature/issue-5-height-does-not-work branch October 9, 2018 18:10
@bertysentry
Copy link
Collaborator Author

Thank you @ziscloud for merging this.

Now, would you allow me to publish this as version 0.4.0?

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