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

Solve citation bug in chrome #770

Merged
merged 9 commits into from
May 4, 2023
Merged

Conversation

lagartoverde
Copy link
Contributor

No description provided.

@lagartoverde lagartoverde self-assigned this Apr 25, 2023
@lagartoverde lagartoverde marked this pull request as draft April 25, 2023 10:13
@lagartoverde lagartoverde marked this pull request as ready for review April 26, 2023 13:48
@abeforgit
Copy link
Member

taking a second to at least link the jira issue in the pr description would really help reviewers

@abeforgit
Copy link
Member

@abeforgit
Copy link
Member

this needs a companion pr on the plugin repo, on GN, and on RB, to enable the plugin. They can be in draft mode until the editor is released, but otherwise we will forget to add the configuration.

const chromeHacksPlugin = new ProsePlugin({
props: {
decorations(state: EditorState): DecorationSet | undefined {
if (!chrome) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we will need to test edge

@lagartoverde lagartoverde merged commit 440f59d into master May 4, 2023
@lagartoverde lagartoverde deleted the bugfix/citation-bug-in-chrome branch May 4, 2023 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants