Skip to content

Insert buffer-file-name as FILE if pdf-view-mode is active - #5

Open
fapdash wants to merge 1 commit into
jeremy-compostella:masterfrom
fapdash:insert-current-file-name
Open

Insert buffer-file-name as FILE if pdf-view-mode is active#5
fapdash wants to merge 1 commit into
jeremy-compostella:masterfrom
fapdash:insert-current-file-name

Conversation

@fapdash

@fapdash fapdash commented Jul 26, 2021

Copy link
Copy Markdown
  • checks if pdf-view-mode is active
  • if it's active it inserts the file name of the currently loaded pdf as PATTERN

@jeremy-compostella I know the code could be cleaner but I wanted to first check if you are generally interested in this change?

@fapdash
fapdash force-pushed the insert-current-file-name branch from 6eb4db2 to aaddfee Compare July 26, 2021 08:42
@fapdash fapdash changed the title Insert buffer-file-name as PATTERN if pdf-view-mode is active Insert buffer-file-name as FILE if pdf-view-mode is active Jul 26, 2021
@jeremy-compostella

Copy link
Copy Markdown
Owner

I like the idea so go ahead and make it cleaner.

Also, we support both doc-view-mode and pdf-view-mode.

@fapdash
fapdash force-pushed the insert-current-file-name branch from aaddfee to bae4737 Compare July 29, 2021 16:35
- checks if pdf-view-mode is active
- checks if doc-view-mode is active
- if either one is active it inserts the file name of the currently
  loaded document as FILE
- adds tests
- adds test command to README
@fapdash
fapdash force-pushed the insert-current-file-name branch from bae4737 to 13ab125 Compare July 29, 2021 16:40
@fapdash

fapdash commented Jul 29, 2021

Copy link
Copy Markdown
Author

@jeremy-compostella Ready for review.

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.

2 participants