freenode :: #microformats

18 Apr 2017
14:22Loqi[@wrong_travis] Note to self: what email clients support microformats in email signatures? (http://twtr.io/1QmWx5AW2x8)
21:37gRegorLoveFor http://pin13.net/mf2/?id=20170418212550655 I misread initially, didn't see the alt=""
21:37gRegorLoveSo does look like a bug in php-mf2, should just use the blank alt value per &quot;replacing any nested <img> elements with their alt attribute, if present; otherwise their src attribute, if present, adding a space at the beginning and end, resolving any relative URLs, and removing all leading/trailing whitespace.&quot;
21:38gRegorLoveThis line should be handling it: https://github.com/indieweb/php-mf2/blob/master/Mf2/Parser.php#L413
21:39aaronpkis the alt attribute being removed because it&#39;s empty?
21:39aaronpklike, before this code gets to it?
21:40gRegorLovePossibly
21:40gRegorLoveHistorically I don&#39;t think so, but I&#39;d have to test.
21:57gRegorLoveHah. Always check for the simplest explanation. Last release was 2016-03-14 and the fix was in 2016-04-17
21:57gRegorLoveWorks with latest: http://gregorlove.com/php-mf2/test.php?id=14
22:01aaronpkoho
22:01aaronpkwell then
22:02gRegorLoveWould really like to get my latest PR in, if it will start passing PHP7 nightly build
22:03gRegorLoveBut regardless, probably time for a new release :)
22:04aaronpkyes i&#39;ve been meaning to check that out
19 Apr 2017
No messages
   
Last message: 37 days and 21 hours ago