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

a bunch of feedback on the beta #2

Closed
chrisgrieser opened this issue Sep 9, 2021 · 3 comments
Closed

a bunch of feedback on the beta #2

chrisgrieser opened this issue Sep 9, 2021 · 3 comments

Comments

@chrisgrieser
Copy link

@chrisgrieser chrisgrieser commented Sep 9, 2021

Here a bunch of smaller issues I discovered while testing

feature feedback

  • the "all commands" commands is a great idea, some other plugins would benefit from doing something like this
  • I am not sure what the usecase for "select previous block" would be, when you already have "select current block" and "expand to previous/next block"? If there is no usecase, you could maybe ditch those to reduce the number of commands?
  • I do not really like the implementation of the pull/push/copy commands. Selecting a note, selecting a beginning line, selecting an ending line... It feels a bit tedious, and I got the impression that working with copy/cut-paste and two panes would be quicker than doing it with this plugin. The "bookmarks" setting is a nice solution to speed up the selection of a note – maybe you could just use a value defined in the options (top of the file, bottom of the file, etc) to skip the selection of beginning/ending lines and so on? Like I press only one hotkey, select out of my 5 bookmarked notes, and the current line gets simply appended/prepended.
  • I get that bookmark+TOP/ BOTTOM does something similar, but I believe it feels a bit counterintuitive. And also means that until you use the plugin with some bookmarks, it would feel tedious. I dunno. Maybe just having a list of all lines is a bit too much, and having TOP/BOTTOM/Headers as options would reduce the tediousness a bit?

wording

  • "copy block as block as block reference" is phrased a bit weird. I'd expect sth like "Copy embedded block reference"?
  • is there a reason it is called "select current line", but also "select next block"? Confuses me a bit, since I'd expect something like "next line" and "current block" as well then. (I never really understood the difference between lines and blocks in Obsidian, tbh.)
  • it only becomes clear upon testing that "copy block as block as block reference" and "Add block ref ID's to selection and Copy them to clipboard" do the same thing. Maybe it could also be phrased as "line/selection" as the other commands?
  • I think the "Alias Indicator" in the settings should rather be called "Alias Placeholder", if I am not misunderstanding the feature?
  • is it necessary to add ; to the end of every bookmark line, when not using TOP/BOTTOM/Heading? (add some info on that?)

Otherwise, as far as I can tell from testing this plugin a bit, I didn't encounter any bugs. (But I also could not test all features, since I do not use daily notes, for example.)

@TfTHacker
Copy link
Owner

@TfTHacker TfTHacker commented Sep 10, 2021

Awesome Awesome! Thank you so much for your help with testing and feedback. I agree, some aspects of the navigation feel tedious. I will do some experimenting on the flow. It is so hard to figure out the right balance of being fast, and having precision control.

will also improve the wording based on your feedback.

@TfTHacker
Copy link
Owner

@TfTHacker TfTHacker commented Sep 10, 2021

Just to mention, “ is it necessary to add ; to the end of every bookmark line, when not usingTOP/BOTTOM/Heading? (add some info on that?)”

you don’t need to use the semi-colon in this case. I will update the description.

@TfTHacker
Copy link
Owner

@TfTHacker TfTHacker commented Sep 11, 2021

I think I have tweaked some of these issues in an upcoming release. Others might wait while we do more experiments with the flow of the tool.

@TfTHacker TfTHacker closed this Sep 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants