Merge pull request #3 from rca-umb/text

Text

Author: rca-umb

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.0] - 2025-1-25
+
+### Removed
+
+- Option to change the text content of notifications.
+
 ## [1.0.1] - 2024-12-30
 
 ### Fixed

README.md

@@ -36,4 +36,4 @@ Show only the first heading: [[Title#Heading#Subheading#Subsubheading|Heading]]
 Show only the last heading: [[Title#Heading#Subheading#Subsubheading|Subsubheading]]
 ```
 
-By default, the headings in the display text will be separated by a single space, but this can be changed to whatever you prefer. Some examples may be a comma (, ), colon (: ), or arrow (-> ). Just not that whatever is typed in the separator text box in the settings will be exactly what is used in the display text, nothing is added to it or removed from it.
+By default, the headings in the display text will be separated by a single space, but this can be changed to whatever you prefer. Some examples may be a comma (, ), colon (: ), or arrow (-> ). Just note that whatever is typed in the separator text box in the settings will be exactly what is used in the display text, nothing is added to it or removed from it.

main.ts

@@ -4,15 +4,13 @@ interface AnchorDisplayTextSettings {
 	includeNoteName : string;
 	whichHeadings: string;
 	includeNotice: boolean;
-	noticeText: string;
 	sep : string;
 }
 
 const DEFAULT_SETTINGS: AnchorDisplayTextSettings = {
 	includeNoteName: 'headersOnly',
 	whichHeadings: 'allHeaders',
 	includeNotice: false,
-	noticeText: 'Link changed!',
 	sep: ' '
 }
 
@@ -48,15 +46,16 @@ export default class AnchorDisplayText extends Plugin {
 						}
 					}
 					const startIndex = (match.index ?? 0) + match[0].length - 2;
-					if (this.settings.includeNoteName === 'headersOnly') {
-						editor.replaceRange(`|${displayText}`, {line: cursor.line, ch: startIndex}, undefined, 'headerDisplayText');
-					} else if (this.settings.includeNoteName === 'noteNameFirst') {
-						editor.replaceRange(`|${headings[0]}${this.settings.sep}${displayText}`, {line: cursor.line, ch: startIndex}, undefined, 'headerDisplayText');
+					// add note name to display text if wanted
+					if (this.settings.includeNoteName === 'noteNameFirst') {
+						displayText = `${headings[0]}${this.settings.sep}${displayText}`;
 					} else if (this.settings.includeNoteName === 'noteNameLast') {
-						editor.replaceRange(`|${displayText}${this.settings.sep}${headings[0]}`, {line: cursor.line, ch: startIndex}, undefined, 'headerDisplayText');
+						displayText = `${displayText}${this.settings.sep}${headings[0]}`;
 					}
+					// change the display text of the link
+					editor.replaceRange(`|${displayText}`, {line: cursor.line, ch: startIndex}, undefined, 'headerDisplayText');
 					if (this.settings.includeNotice) {
-						new Notice (this.settings.noticeText)
+						new Notice (`Updated anchor link display text.`);
 					}
 				}
 			})
@@ -79,24 +78,16 @@ export default class AnchorDisplayText extends Plugin {
 
 class AnchorDisplayTextSettingTab extends PluginSettingTab {
 	plugin: AnchorDisplayText;
-	private notificationTextSetting?: Setting;
 
 	constructor(app: App, plugin: AnchorDisplayText) {
 		super(app, plugin);
 		this.plugin = plugin;
 	}
 
-	displayNotificationTextSetting() {
-        if (this.notificationTextSetting) {
-            this.notificationTextSetting.settingEl.style.display = this.plugin.settings.includeNotice ? 'flex' : 'none';
-        }
-    }
-
 	display(): void {
 		const {containerEl} = this;
 		containerEl.empty();
 
-		new Setting(containerEl).setName('Display text format').setHeading();
 		new Setting(containerEl)
 			.setName('Include note name')
 			.setDesc('Include the title of the note in the display text.')
@@ -134,28 +125,15 @@ class AnchorDisplayTextSettingTab extends PluginSettingTab {
 				});
 			});
 
-		new Setting(containerEl).setName('Notifications').setHeading();
 		new Setting(containerEl)
 			.setName('Enable notifications')
-			.setDesc('Have a notice pop up whenever a link is automatically changed.')
+			.setDesc('Have a notice pop up whenever an anchor link is automatically changed.')
 			.addToggle(toggle => {
 				toggle.setValue(this.plugin.settings.includeNotice);
 				toggle.onChange(value => {
 					this.plugin.settings.includeNotice = value;
-					this.displayNotificationTextSetting();
-					this.plugin.saveSettings();
-				});
-			});
-		this.notificationTextSetting = new Setting(containerEl)
-			.setName('Notification text')
-			.setDesc('Set the text to appear in the notification.')
-			.addText(text => {
-				text.setValue(this.plugin.settings.noticeText);
-				text.onChange(value => {
-					this.plugin.settings.noticeText = value;
 					this.plugin.saveSettings();
 				});
 			});
-		this.displayNotificationTextSetting();
 	}
 }

manifest.json

@@ -1,7 +1,7 @@
 {
 	"id": "anchor-display-text",
 	"name": "Anchor Link Display Text",
-	"version": "1.0.1",
+	"version": "1.1.0",
 	"minAppVersion": "0.15.0",
 	"description": "Automatically uses the linked heading as the display text for the anchor links.",
 	"author": "Robert C Arsenault",

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "anchor-display-text-plugin",
-	"version": "1.0.1",
+	"version": "1.1.0",
 	"description": "Plugin for Obsidian (https://obsidian.md) that sets the display text of anchor links to the name of the heading.",
 	"main": "main.js",
 	"scripts": {