Add plugin: Anchor Link Display Text

[rca-umb]

I am submitting a new Community Plugin

Repo URL

Link to my plugin: https://github.com/rca-umb/anchor-link-display-text

Release Checklist

• I have tested the plugin on
  • Windows
  • macOS
  • Linux
  • Android (if applicable)
  • iOS (if applicable)
• My GitHub release contains all required files (as individual files, not just in the source.zip / source.tar.gz)
  • main.js
  • manifest.json
  • styles.css (optional)
• GitHub release name matches the exact version number specified in my manifest.json (Note: Use the exact version number, don't include a prefix v)
• The id in my manifest.json matches the id in the community-plugins.json file.
• My README.md describes the plugin's purpose and provides clear usage instructions.
• I have read the developer policies at https://docs.obsidian.md/Developer+policies, and have assessed my plugins's adherence to these policies.
• I have read the tips in https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines and have self-reviewed my plugin to avoid these common pitfalls.
• I have added a license in the LICENSE file.
• My project respects and is compatible with the original license of any code from other plugins that I'm using.
  I have given proper attribution to these other projects in my README.md.

[ObsidianReviewBot]

The automated review has not found any issues with your code, assigning human for additional review.

[Zachatoo]

This is an awesome plugin idea!

---

.setName('Notification text')
Minor UX critique, I would recommend conditionally showing this setting based on the previous setting. It isn't relevant to the user if the previous setting is disabled.

[ObsidianReviewBot]

Changes requested by Zachatoo have been made, assigning additional human for additional review

[rca-umb]

This is an awesome plugin idea!

.setName('Notification text') Minor UX critique, I would recommend conditionally showing this setting based on the previous setting. It isn't relevant to the user if the previous setting is disabled.

Good point, I appreciate the feedback!

Edit: Fixed in 1.0.1

[joethei]

this.notificationTextSetting = new Setting(containerEl)
Why should this be a setting?
It makes no sense for me.

[rca-umb]

this.notificationTextSetting = new Setting(containerEl) Why should this be a setting? It makes no sense for me.

@joethei
That's a fair question. I suppose I just wanted to give maximum flexibility. I definitely wanted the option for a notification because I personally like being notified when things get changed automatically, but I could see some people finding it unnecessary. When it came to the actual contents of the message, I wasn't sure how verbose it should be; "link changed", "anchor link changed", "foo#bar changed to foo#bar|foo bar", etc. For me, I don't really care what it says as long as I can see the visual feedback when an edit is programmatically made to my document, so I just figured I would give the user the option in case something worked better for them. I understand that this may not be the best choice and can make it fixed.