Skip to content

Remove the need to parse the Markdown to extract the markdown title#5

Merged
StefanAndreevHX merged 2 commits intomainfrom
feature/extract-the-markdown-title-image-links
Jan 19, 2026
Merged

Remove the need to parse the Markdown to extract the markdown title#5
StefanAndreevHX merged 2 commits intomainfrom
feature/extract-the-markdown-title-image-links

Conversation

@StefanAndreevHX
Copy link
Copy Markdown

  • Add MarkdownLinkClickConfiguration with url, title, and isImage properties
  • Add openMarkdownLink environment value for custom link handling
  • Add URL extension helpers for encoding/extracting titles in text links
  • Update ImageView to pass title through LinkModifier
  • Add LinksView demo showing both image and text link handling

…ment for accessing link title at click time

- Add MarkdownLinkClickConfiguration with url, title, and isImage properties
- Add openMarkdownLink environment value for custom link handling
- Add URL extension helpers for encoding/extracting titles in text links
- Update ImageView to pass title through LinkModifier
- Add LinksView demo showing both image and text link handling
Copy link
Copy Markdown
Collaborator

@dpetrov-appolica dpetrov-appolica left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Tsvetan11
Tsvetan11 previously approved these changes Jan 13, 2026
}
.environment(\.openMarkdownLink) { configuration in
.environment(\.openMarkdownLink) { [self] configuration in
print("Handler called: \(configuration.title)")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If those prints were only for debugging purpose, remove them.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added them for debugging, to prove/debug if it work.

@StefanAndreevHX StefanAndreevHX merged commit fb5d01b into main Jan 19, 2026
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.

3 participants