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

[CLOSED] Quick Edit ignores everything in file after selector that begins with ":" #8753

Open
core-ai-bot opened this issue Aug 30, 2021 · 6 comments

Comments

@core-ai-bot
Copy link
Member

Issue by rcbraggthree
Thursday Nov 06, 2014 at 20:34 GMT
Originally opened as adobe/brackets#9840


When I try and use quick edit in my document to edit a class or id it works only if the rule is above a ::selection or ::after. If the class or id is after any of these rules that include a : or - then it will not find it and it wants me to create a new rule.

windows 8.1
Release 1.0 build 1.0.0-15191 (release 3f2eb90a4)

::selection { background-color: #67b2e7; color: #fff; }
::selection {color:#fff;background: #5a7388;}::-moz-selection {color:#fff;background: #5a7388;}

@core-ai-bot
Copy link
Member Author

Comment by RaymondLim
Thursday Nov 06, 2014 at 21:29 GMT


@rcbthree I can't reproduce it with my own::selection. Can you provide the CSS code that is causing the issue by pasting it in a comment or at the end of your description? Below is the CSS code that I tried.

.myh2::selection {
    color: white;  background: black;
}

.myh2 {
    margin-top: 1.3em;
}

@core-ai-bot
Copy link
Member Author

Comment by RaymondLim
Monday Nov 17, 2014 at 00:53 GMT


Raise it to high priority since we have another related issue #9942 and it can crash Brackets after closing the developer tools.

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Monday Nov 17, 2014 at 21:40 GMT


The dev tools crash is an unrelated issue, so I think this should go back down to Medium (sort of moot though, since there's already a PR up).

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Tuesday Nov 18, 2014 at 01:23 GMT


Confirmed. Closing.

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Wednesday Dec 17, 2014 at 21:06 GMT


Fix PR includes a unit test, so removing 'needs unit test'

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Wednesday Dec 17, 2014 at 21:14 GMT


Clarifying title

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

No branches or pull requests

1 participant