Skip to content
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

Jump to bottom

Link HTML messed up in Advisor #16621

Closed
nunoperalta opened this issue Feb 6, 2021 · 2 comments · Fixed by #16623
Closed

Link HTML messed up in Advisor #16621

nunoperalta opened this issue Feb 6, 2021 · 2 comments · Fixed by #16623
Assignees
@MauricioFauth
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete
Projects
issues
Milestone
5.1.0

Comments

@nunoperalta
Copy link

Where it says "this blog entry", it appears it should be 2 links instead of 1, and the HTML for the second link is being seen as a whole link (possibly a problem with escaping):

image

Version 5.0.4
@ibennetch
Copy link
Member

Thanks for the report. This is indeed incorrect, and the problem continues in master.

@ibennetch ibennetch mentioned this issue Feb 6, 2021
Closed
@MauricioFauth MauricioFauth self-assigned this Feb 6, 2021
@MauricioFauth MauricioFauth mentioned this issue Feb 6, 2021
Merged
@MauricioFauth MauricioFauth added the Bug A problem or regression with an existing feature label Feb 6, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Feb 6, 2021
@williamdes williamdes added this to the 5.1.0 milestone Feb 6, 2021
@williamdes williamdes added this to Needs triage in issues via automation Feb 6, 2021
@williamdes williamdes moved this from Needs triage to ready to merge in issues Feb 6, 2021
@williamdes williamdes linked a pull request Feb 8, 2021 that will close this issue
Fix link HTML messed up in Advisor #16623
Merged
@williamdes
Copy link
Member

Should be fixed now, in next 5.1 and on master

issues automation moved this from ready to merge to Closed Feb 8, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MauricioFauth MauricioFauth

Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete
Projects
issues
  
Closed
Milestone
5.1.0
Development

Successfully merging a pull request may close this issue.

Fix link HTML messed up in Advisor MauricioFauth/phpmyadmin
4 participants
@MauricioFauth @nunoperalta @ibennetch @williamdes