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

Automatic selection is buggy, hinders easy selection, double .. are the cause #288

Closed
cantidecidelater opened this issue Oct 29, 2013 · 1 comment
Labels
bug Bug report
Milestone

Comments

@cantidecidelater
Copy link

When clicking on elements that have multiple classes, the Stylebot naively copy-pastes the class tag and replaces the spaces with dots like, class="clsA clsB" becomes .clsA.clsB

This becomes a problem when there are multiple spaces separating classes in markup. For example, on medium.com, when clicking the background, Stylebot selects it like this:
section.post-page-wrapper..post-page-wrapper-contain

when in fact, it should be like this:
section.post-page-wrapper.post-page-wrapper-contain

The double dots between post-page-wrapper and post-page-wrapper-contain cause the style to fail to apply, requiring the additional effort to manually edit the selector and delete any double periods. Not a biggie for who knows about it, but this is counter-intuitive.

Collapsing the spaces before replacing them with . might be a solution.

@ankit
Copy link
Owner

ankit commented Nov 27, 2013

Thanks @cantidecidelater for reporting this issue! I will put in a fix in the next update.

@ankit ankit closed this as completed in f7d2bfc Dec 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report
Projects
None yet
Development

No branches or pull requests

2 participants