Adjust CSS styling of rich text toolbar to Bootstrap 4 for consistency

[jywarren]

I think the lines of code could be somewhere around here?

PublicLab.Editor/src/adapters/PublicLab.Woofmark.js

Line 372 in cf1fc4a

$(".wk-switchboard button").addClass("btn-sm");

See how the buttons here don't match -- i think the basic buttons are via woofmark but the black-outlined ones are the ones we've added programmatically after woofmark is loaded. We'd love help with this, it probably will take some trial and error but isn't such a complex change!

Originally posted by @jywarren in #663 (comment) (cc @keshav234156, noting I've split this out!)

[eynglv]

@jywarren hi, could i take this bug? would be my first open source contribution

are you looking to change the outline (make all the buttons outlined) or you want them to be basic (via woofmark)?

[devan5hu]

@jywarren Hello! I am fairly new to contributing to major repositories on Github in-fact this is my very first comment. I started working on this issue today and found out the fix for it. Just wanted to share a picture of the result before I created a new branch. The changes were very minor and I want to know if its correct!

I also fixed the Custom Insert button in responsive view.

Please correct me if I am wrong and you expected something else!

[Goodiec]

Hi, @jywarren, I'm Goodness, an Outreachy applicant interested in contributing to the "Refine the Public Lab Rich Editor" project. I would like to if this issue is still open and if I can work on it.

[jywarren]

Hi @Goodiec we'd love a contribution here. I wonder if #705 already gets at part of it? However i wonder if it could be done more efficiently in a single CSS selector instead of individually for each.

I'm also not sure if this recent tweak I made may have resolved the issue as well?

b1cb346

But, if you can try it out locally with the latest code to confirm, we should be able to close this issue one way or another. Thank you!!!

[Goodiec]

Hi @Goodiec we'd love a contribution here. I wonder if #705 already gets at part of it? However i wonder if it could be done more efficiently in a single CSS selector instead of individually for each.

I'm also not sure if this recent tweak I made may have resolved the issue as well?

b1cb346

But, if you can try it out locally with the latest code to confirm, we should be able to close this issue one way or another. Thank you!!!

Sure, I'll try that. Thank you!

[RaviAnand111]

hey @jywarren @TildaDares @NARUDESIGNS, can I work on this issue ?

[TildaDares]

@RaviAnand111 I think this issue has been resolved. I checked on my local environment and all of the buttons match.

[TildaDares]

Thanks @RaviAnand111! I think we can close this issue now.

[RaviAnand111]

Hey @TildaDares, I checked my local environment and all the buttons do not match,

Do you know why is this happening?

[TildaDares]

@RaviAnand111 Have you pulled changes from this repo recently?

[RaviAnand111]

yes I did, git status, says uptodate see