-
Notifications
You must be signed in to change notification settings - Fork 123
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
Show pointer and use underline dashed for click trigger #1111
Conversation
A regression from MarkBind#1033 Let's add back the two styles of triggers: - click: show mouse pointer to signal that it is clickable and use underline dashed - not click: no special cursor, and use underline dotted
Seems to work fine 👍 Might be good to change the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
do resolve the minor test file conflict though! @openorclose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…bind into remove-fixed-bugs * 'remove-fixed-bugs' of https://github.com/Tejas2805/markbind: Docs: Add contributing.md (MarkBind#1139) Show pointer and use underline dashed for click trigger (MarkBind#1111) Support variables to be defined as by JSON (MarkBind#1117) Allow an array for specifying page src or glob (MarkBind#1118) Code blocks: Implement inline markdown support for heading (MarkBind#1137) Fix lazy reload for urls without index (MarkBind#1110) Changes remaining references from filterTags to tags (MarkBind#1149)
Fix a regression from #1033. Let's add back the two styles of triggers: - click: show mouse pointer to signal that it is clickable and use underline dashed - not click: no special cursor, and use underline dotted
@openorclose this doesn't seem to be working in the latest release. This hover tooltip is shown as a dashed underline (should be a dotted underline?) |
What is the purpose of this pull request? (put "X" next to an item, remove the rest)
• [X] Bug fix
Fixes #1053
What is the rationale for this request?
Regression from #1033, the migration to bootstrap-vue.
There were two styles for triggers depending on if it was a
click
trigger or not.During the migration, only one style was used for all triggers.
What changes did you make? (Give an overview)
Implemented the two different styles depending on type of trigger.
Provide some example code that this change will affect:
Is there anything you'd like reviewers to focus on?
Testing instructions:
Check that click triggers have the proper style
Proposed commit message: (wrap lines at 72 characters)