-
Notifications
You must be signed in to change notification settings - Fork 14
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
Right border not showing #25
Comments
Hi @martinmanzo, I just wanna know why you are using the property |
Hi @JulioWar, thanks for the response. I had some Issues getting the switch to display correctly and turned into old issues to now if someone else had had the same problem and if there was a solution. I found this issue #2 and in an answer you state to try with that. I did it and the switch started to display correctly. |
Ok, I understand. Can you please make the pull request I gonna take the time to test that on my side. Thanks |
Sure, I'll do it in the next days. |
This is still an issue. Fix that @martinmanzo suggested seems to work. |
Hi @hashmed This an issue that I can't reproduce and I don't know if I make this changes, others can be affected. people using this package that doesn't have this issue. That is the case for me. Can you tell me if you see the issue there ? Because that demo is based on the same code and on my side I don't see the issue Thanks in advance for using my package 👍 |
@JulioWar I do not see the issue on demo site, border is there. I'm now suspecting that some other css files might be interfering with it. I will let you know if I pinpoint the issue. But your library is probably good as it is. :) |
Bug Report or Feature Request (mark with an
x
)Versions.
2.4.10
Repro steps.
<bSwitch
[switch-base-class]="'bootstrap-switch'"
[switch-wrapper-class]="'bootstrap-switch-wrapper'"
[switch-off-text]="'Excluir'"
[switch-on-text]="'Incluir'"
[switch-size]="'small'"
[switch-disabled]="false"
[switch-readonly]="false"
[switch-animate]="true"
[switch-inverse]="false"
[switch-handle-width]="60"
[ngModel]="inc"
(onChangeState)="modifyFilterInclusive($event, fld)">
and this is the result:
![image](https://user-images.githubusercontent.com/3625887/34951047-bae84a36-f9f3-11e7-952c-bbab2fffebd5.png)
The log given by the failure.
None
Desired functionality.
Adding 2px to the width is enough to show the border, example:
![image](https://user-images.githubusercontent.com/3625887/34951102-eb41de18-f9f3-11e7-91da-d17e9a7be18a.png)
Mention any other details that might be useful.
By looking at the source code the change should be made to this line:
https://github.com/JulioWar/jw-bootstrap-switch-ng2/blob/master/src/directive.ts#L32
<div class="{{ getWrapperClasses() }}" [style.width]=" (handleWidth + labelWidth ) +'px'">
and transform it into:
<div class="{{ getWrapperClasses() }}" [style.width]=" (handleWidth + labelWidth + 2) +'px'">
I could do a pull request if you need.
Thanks!
The text was updated successfully, but these errors were encountered: