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

added inCode flag #8

Closed
wants to merge 2 commits into from

Conversation

bobzel
Copy link
Contributor

@bobzel bobzel commented Jan 29, 2024

added inCode flag to inputRule specs to allow inputRules to selectively apply to code blocks.

added inCode flag to inputRule specs to allow inputRules to selectively apply to code blocks.
@@ -35,12 +36,17 @@ export class InputRule {
/// When set to false,
/// [`undoInputRule`](#inputrules.undoInputRule) doesn't work on
/// this rule.
undoable?: boolean
undoable?: 'boolean',
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this should be a string.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, you're right, don't know how I changed that.

@@ -11,6 +11,7 @@ export class InputRule {

/// @internal
undoable: boolean
inCode?: 'allowed' | 'only';
Copy link
Member

Choose a reason for hiding this comment

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

Could you make this boolean | "only", so people can also explicitly provide false?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

@bobzel
Copy link
Contributor Author

bobzel commented Jan 30, 2024

Just curious since this is my first pull request -- how long should I expect it to take to have it merged? days, weeks, months?
Thanks and sorry for being so naive to this process.

@marijnh marijnh closed this in a993c59 Jan 30, 2024
@marijnh
Copy link
Member

marijnh commented Jan 30, 2024

Merged as 66d47ea, tidied up in a993c59. Released as part of prosemirror-inputrules 1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants