Skip to content

Added title formatting for the link template#3

Open
auxiliary-character wants to merge 1 commit into
eganwall:mainfrom
auxiliary-character:main
Open

Added title formatting for the link template#3
auxiliary-character wants to merge 1 commit into
eganwall:mainfrom
auxiliary-character:main

Conversation

@auxiliary-character

Copy link
Copy Markdown

I added a # character to the start of the link template, which should
cause reddit's markdown formatting to format the links as a title,
causing them to be bigger and even easier to click on when on mobile.

Yes. A singular character difference. lol

I added a `#` character to the start of the link template, which should
cause reddit's markdown formatting to format the links as a title,
causing them to be bigger and even easier to click on when on mobile.
@eganwall

eganwall commented Jan 2, 2022

Copy link
Copy Markdown
Owner

Hey aux, thanks for the PR. The lack of title formatting in the link reposts is an intentional design decision - I wanted to try and strike a balance between usefulness and obtrusiveness, and I think an H1 heading for each link would make the bot's comments too large, eye-catching, and annoying. Maybe something like an H4 heading (####) would be a good compromise?

So it would be

H1 heading

vs.

H4 heading

I think an H4 accomplishes the goal of making the links slightly bigger and easier to find/click, without making the comment too obtrusive. Thoughts?

@auxiliary-character

auxiliary-character commented Jan 3, 2022

Copy link
Copy Markdown
Author

That's a good point about the balance between usefulness and obtrusiveness. I kind of figured it just was something you hadn't considered.

That said, I'm not sure H4 would be appreciably larger than unformated text? The idea is to make the clickable area for the text larger so it's easier to click, and if the text is essentially the same size, but bold, then that's kind of a step backward - being more obtrusive, without increasing usefulness. I would think maybe an H3 or H2 would be appropriate, but if not, then maybe just leave it as is.

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