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

Highlight emphasis/strong at the end #204

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Conversation

Waqar144
Copy link
Contributor

@Waqar144 Waqar144 commented Mar 21, 2024

Should fix pbek/QOwnNotes#2952
Should fix pbek/QOwnNotes#2972

Didn't test it a lot so there might still be issues

Should fix QOwnNotes #2952

Didn't test it a lot so there might still be issues
@pbek
Copy link
Owner

pbek commented Mar 21, 2024

Thank you very much!
But I tested the example in QOwnNotes:

```
(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\[(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|[a-z0-9-]*[a-z0-9]:(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21-\x5a\x53-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])+)\])
```

    (?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\[(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|[a-z0-9-]*[a-z0-9]:(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21-\x5a\x53-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])+)\])

`(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\[(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|[a-z0-9-]*[a-z0-9]:(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21-\x5a\x53-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])+)\])`

And it still showed my note folder URL in the preview...

@pbek
Copy link
Owner

pbek commented Mar 21, 2024

image

@Waqar144
Copy link
Contributor Author

Yeah I am not sure if fixing those is very practical atm given there is no way to know about regressions and they seem like extreme cases

@pbek
Copy link
Owner

pbek commented Mar 21, 2024

Yeah, would have regressions for sure! 😬

Which markdown should I test your PR against?

@Waqar144
Copy link
Contributor Author

Linked the wrong ticket, it is addressing: pbek/QOwnNotes#2972
So the markdowns in that ticket should be okay now.

@pbek
Copy link
Owner

pbek commented Mar 21, 2024

No, you linked pbek/QOwnNotes#2952, didn't you? 😅

Linked the wrong ticket...

Ah, sorry!

@pbek pbek merged commit 35330ea into pbek:develop Mar 21, 2024
21 of 22 checks passed
@pbek
Copy link
Owner

pbek commented Mar 21, 2024

I really dislike the automatic issue closing, when you merge a PR. Both issues were closed now.
https://github.com/orgs/community/discussions/23476 🙄

@Waqar144
Copy link
Contributor Author

At least it shouldn't work if the repo is different lol. Does this mean we can close issues anywhere on github?

@pbek
Copy link
Owner

pbek commented Mar 21, 2024

I guess not. I closed it when I merged the PR... But there is no way around that, maybe one needs to remove the link to the issue from the first (?) comment...

@Waqar144
Copy link
Contributor Author

probably

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

Successfully merging this pull request may close these issues.

[BUG] Regex preview [BUG] Text between two wildcards in code block in bullet point list gets stylized
2 participants