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

Fix Multicursor Paste Again and Enhance Paste Modification Logic by Only Enabling when A Paste Rule is Enabled #879

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

pjkaufman
Copy link
Collaborator

Relates to #480

Changes Made:

  • When I made some changes to fix an issue with pasting checklist and list values, it broke multi-cursor paste again, so that needed a fix for adding the selected line content
  • Updated the paste without modification rule to just go ahead and only show when paste overriding is active
  • Paste overriding will now only be active when a paste rule is enabled
  • Options now call out to the plugin method for saving settings instead of just hitting save data directly

…de sure that multi-cursor paste got fixed for handling the current line content, made sure that paste is only affected when a paste rule is enabled, and made sure that the paste as plaintext option only shows when paste override is enabled
@pjkaufman pjkaufman self-assigned this Sep 6, 2023
@pjkaufman pjkaufman added bug Something isn't working obsidian Obsidian specific syntax issue or feature labels Sep 6, 2023
@pjkaufman pjkaufman merged commit 054b5e2 into platers:master Sep 6, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working obsidian Obsidian specific syntax issue or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant