Web forum is in read-only mode. Login as active registered customer for write access
  Forum Search   New Posts New Posts

TAdvMemo and URL highlight bug (again)

 Post Reply Post Reply
Author
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Topic: TAdvMemo and URL highlight bug (again)
    Posted: 19 Apr 2019 at 12:34am
I thought I'd give TAdvMemo another try after reporting a lot of bugs and I hoped they would be fixed by now but unfortunately not.

This is really frustrating.

See the screenshot where the URL highlighting is messed up because the words "for" and "displayed" and "information." are highlighted like they are a URL.

I still have not tested to see if the high DPI bugs have been fixed....

It is really frustrating to pay for these components and not to be able to use them even after reporting these issues months ago.

Screenshot:
https://www.htmlvalidator.com/bugs/TAdvMemo/bug.png

Can this bug finally be eradicated?

Thank you.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 19 Apr 2019 at 12:36am
And when I click on the link with "F30" in it, it tries to take me to:

When it should take me to:

Not sure why it is appending the "for" to the end of the link when there is a space character in front of it.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 19 Apr 2019 at 2:32am
By the way, here is the text contained in the TAdvMemo:
Ensure that images with empty or null alt text and no title attribute are decorative or should otherwise be ignored by assistive technology [A, 1.1.1; F30, H67] [A, 1.2.1; F30]. Visit https://www.w3.org/TR/WCAG20-TECHS/F30 and https://www.w3.org/TR/WCAG20-TECHS/H67 for more information. This message is displayed up to 3 times.
---                            
(Accessibility) Images may require a "longdesc" attribute to provide a URI that contains a long description that describes the image [A, 1.1.1; H45]. Visit https://www.w3.org/TR/WCAG20-TECHS/H45 for more information. This message is displayed up to 3 times.
---
Image searches can be a significant source of visitors and traffic. Therefore, try to use keywords when naming images, and separate keywords with hyphens (not underscores). This message is displayed only once.
---
[7] Ensure that the "img" element is rarely, if ever, used as a layout tool. Always use CSS for layout and positioning. This message is displayed only once.
---
GIF images are well supported in today's browsers but so are PNG images, which are, in many cases, superior to GIF because PNG images can support more colors, better transparency, and better compression (smaller file sizes). However, PNG does not support animation and may suffer from CSS color matching issues (visit https://www.htmlvalidator.com/CSEForum/viewtopic.php?f=3&t=1583). Consider using PNG images instead of GIF images if animation is not required. JPG is usually best for photographs. This message is displayed only once. There may be other GIF images specified in this document.
---
(Accessibility) If using an animated GIF (or other animated or updating content), then be sure that it is accessible. For example, make sure that content that blinks for more than five seconds has a mechanism to pause. Also, set animated gif images to stop blinking within 5 seconds. [A, 2.2.2; F7, G152]. Visit for https://www.w3.org/TR/WCAG20-TECHS/F7 and https://www.w3.org/TR/WCAG20-TECHS/G152 more information. This message is displayed only once.
---
6 messages for line 12
Back to Top
Fare' Daniele View Drop Down
Senior Member
Senior Member
Avatar

Joined: 30 Dec 2015
Posts: 167
Post Options Post Options   Quote Fare' Daniele Quote  Post ReplyReply Direct Link To This Post Posted: 19 Apr 2019 at 10:46am
Good morning Albert,
reading your post i tried to made some test.
So i open Delphi 10.2 and, later, 10.3.1, new project, insert in a new form and only one tadvmemo with default setting.
I made 3 test, after inserting your above text, and the result are:
1 - With no wordwrap the text is handled in a rigth way, all link work in a rigth way
2 - With wordwrap setted to rigth margin, as default 80, the text is handled in a rigth way, the links don't work as expected in details
A) https://www.w3.org/TR/WCAG20-TECHS/H45 it's does not work because when you click on it the link show in chrome is https://www.w3.org/TR/WCAG20-TECHS/H45displayed
B) https://www.w3.org/TR/WCAG20-TECHS/F30 does not work because the link advmemo pass to browser is
https://www.w3.org/TR/WCAG20-TECHS/F30https://www.w3.org/TR/WCAG20-TECHS/H67
(thake also the second link)
C) https://www.w3.org/TR/WCAG20-TECHS/H67 does not work beacuse the browser try to open https://www.w3.org/TR/WCAG20-TECHS/H67displayed
D) https://www.w3.org/TR/WCAG20-TECHS/H45 as above, the link passed is
https://www.w3.org/TR/WCAG20-TECHS/H45displayed

Ther's the word "displayed" that caming from somewhere (not from the clipboard) that broke the component job. Also if there are 2 link close, the component see only one that is the link1 + link2.

3 - With wordwrap setted to client margin, dependig to component width, in this case i got all the errors you described and the links passed to browser are very dirthy, all are broken.
This is a serious problem.

So how to solve the problem?
This is a "non elegant" solution but one is better then zero.
Temporary, and if it's a non massive form rearrange, set the wordwrap to wwnone, increase advmemo width in order to see the maximum text and let the customer scroll the text with the bar. Becuse this is the only way the links work.
As told befor, it's not a good solution you expected, but till the team does not fix the issue i don't see any other solution.
Also, i hope gave to team a good way to fix the problems.

Have a good Easter

Daniele
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 19 Apr 2019 at 3:45pm
Hi Daniele,

Thank you for confirming this problem/bug and suggesting a workaround. Like you said, the workaround is not good.

I'll just continue to use another component that works. The only downside is that it does not make the links clickable. Hopefully TMS will squash this one because I am pretty darn sure I reported this months ago and I thought it had been fixed.

Back to Top
Bruno Fierens View Drop Down
TMS Support
TMS Support
Avatar

Joined: 11 May 2010
Posts: 8601
Post Options Post Options   Quote Bruno Fierens Quote  Post ReplyReply Direct Link To This Post Posted: 20 Apr 2019 at 1:55pm
This must be related to wordwrap and we will reopen this case.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 14 Jun 2019 at 9:58pm
Has this been fixed yet? I have a new release coming up and I'd like to give it another try.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 26 Jun 2019 at 5:27pm
Hi TMS & Bruno,

Well, to answer my own question, I checked it again. TAdvMemo 3.6.5.1.

Still not fixed.

This is very disappointing and discouraging. I still cannot use this component that I paid for and this has been a bug for a long time.

I don't understand how TAdvMemo can have so many features when it still can't properly highlight URLs.


Back to Top
Bruno Fierens View Drop Down
TMS Support
TMS Support
Avatar

Joined: 11 May 2010
Posts: 8601
Post Options Post Options   Quote Bruno Fierens Quote  Post ReplyReply Direct Link To This Post Posted: 27 Jun 2019 at 8:51am
We will reopen this case. We have been overwhelmed lately with work which caused a delay on some thing.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 21 Nov 2019 at 10:39pm
Hello,

The next major release of my app is fast approaching.... and I would really like to use TAdvMemo if it can properly highlight and parse URLs.

Has this been fixed yet? The last I heard (from Bruno Fierens) in late July 2019 was that you guys have been "snowed under in work" and that this bug would be reopened and fixed once & for all.

Thank you.
Back to Top
Bruno Fierens View Drop Down
TMS Support
TMS Support
Avatar

Joined: 11 May 2010
Posts: 8601
Post Options Post Options   Quote Bruno Fierens Quote  Post ReplyReply Direct Link To This Post Posted: 23 Nov 2019 at 10:30am
We can confirm this issue was traced & solved.
The next TMS VCL UI Pack release will have this fix.
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 23 Nov 2019 at 5:49pm
Originally posted by Bruno Fierens

We can confirm this issue was traced & solved.
The next TMS VCL UI Pack release will have this fix.

Great, thank you!
Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 03 Dec 2019 at 4:51pm
I'm testing out the component again and it does look like it will be usable (so far) but I did run into a bug where a link still wasn't highlighted and linked properly (and didn't work when I clicked on it).

The link that was not parsed properly: https://developers.facebook.com/docs/sharing/best-practices#images

It only parsed this much: https://developers.facebook.com/docs/sharing/best-

Also, the forum software gave me some weird 'Access denied' message when I first tried to post this message... and I had to go back and try again and reconstruct this message. Very annoying. Hope it works this time and this time I'm using copy and paste.






Back to Top
Wiersch Albert View Drop Down
Member
Member
Avatar

Joined: 18 Apr 2011
Posts: 52
Post Options Post Options   Quote Wiersch Albert Quote  Post ReplyReply Direct Link To This Post Posted: 03 Dec 2019 at 9:30pm
Sadly I found another bug.

In this case, the URL highlights properly, but when you click on it, it tries to open this link:

When it should open:
https://www.w3.org/TR/WCAG20-TECHS/H45

The text in the memo control is:
Images may require a "longdesc" attribute to provide a URI that contains a long description that describes the image [A, 1.1.1; H45]. Visit https://www.w3.org/TR/WCAG20-TECHS/H45 for more information. This message is displayed up to 3 times.
---
1 message for line 34


Back to Top
Bruno Fierens View Drop Down
TMS Support
TMS Support
Avatar

Joined: 11 May 2010
Posts: 8601
Post Options Post Options   Quote Bruno Fierens Quote  Post ReplyReply Direct Link To This Post Posted: 04 Dec 2019 at 11:13am
We have investigated & improved this. We have sent an incremental source update by email.
Back to Top
 Post Reply Post Reply

Forum Jump Forum Permissions View Drop Down