-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Change close button title / icon #52
Comments
Hi @DominikGanic, isn't this easier with the custom UI? |
Yep, you're right. I'm currently implementing my custom UI. But I have problems with positionating the guide like previously popper did the job. Hm. Gonna need to figure out how I'll do this as it's now static. |
Hey @DominikGanic, I would love to try helping with your case if you want to share your code examples |
Hi @fatihsolhan, watch out the screen below. There are two problems which make it complicated for me tho.
Sadly it's not possible to determine the position of the currently attached element so I can't do anything here with a custom ui. Otherwise, using your UI, which was fine enough, I can't change the close button (icon as well). That makes me sad. :( |
I've managed to let it look like I've wanted. For anybody else, I've attached the CSS belowe which overrides some styles. The SVG with the "X" is also now perfectly visible. :) .v-onboarding-item__header-title { .v-onboarding-item__description { .v-onboarding-item__header-close { &:hover { svg { |
Hi again, it would be great if we could somehow change the closing buttons title or content. As currently there is an SVG which is fine enough - but I'd like to modify the buttons class to set some bootstrap classes like "btn-primary btn-sm" which would give it some colors and style.
Also I'd like to place myself the content instead of SVG I would love to use the char "X" as inner value. It would be much better if we could also put some HTML inside like "" to integrate font awesome icons into the close button.
This would improve the entire behaviour as I'd have to hardcode it now and that's not a proper solution. We're planning to use this within a big spread and great platform for real estate rentals and leasing in germany - so we have to stick to some professional solutions instead of hardcoding anything.
Thanks mate! Hope to hear from you soon! :)
The text was updated successfully, but these errors were encountered: