edbrowse-dev - development list for edbrowse
 help / color / mirror / Atom feed
From: Kevin Carhart <kevin@carhart.net>
To: Adam Thompson <arthompson1990@gmail.com>
Cc: Karl Dahlke <eklhad@comcast.net>, edbrowse-dev@lists.the-brannons.com
Subject: Re: [Edbrowse-dev] questions about new finds
Date: Sat, 2 Jul 2016 15:11:58 -0700 (PDT)	[thread overview]
Message-ID: <alpine.LRH.2.03.1607021453470.8153@carhart.net> (raw)
In-Reply-To: <20160702164833.GA13590@odin>



Hi Adam and Chris,

There's a lot to do basically.  I see tidy errors, but I gravitate to the 
JS errors, I guess partially because they are autonomous for us to fix, 
while messages from tidy involve making threads with the tidy developers, 
fun and worthy in its own right, but the path of least resistance takes 
you to something you can work on right away.  (Dang, here's a toast to the 
tidy group!  It underpins everything now and it's so good I don't think 
about it!  We only integrated tidy around 1 year ago.)

We could add things to availableTags one by one as we find them but this 
will go forever.  I experimented adding a "placeholder" to availableTags 
and editing html.c so that if the call to newTag fails to find a string,
it will create your element as a "placeholder."  But I don't know what I'm 
doing - I was also seeing some infinite loops in renderNode, which I 
probably caused by adding placeholder.  I don't know what the TAGACT or 
flags should be for a made-up placeholder.

K



On Sat, 2 Jul 2016, Adam Thompson wrote:

> Hi all,
>
> First of all appologies for going silent for a while; work, life and computer issues basically.
>
> On Sun, Jun 26, 2016 at 11:39:49PM -0700, Kevin Carhart wrote:
>>
>>> (1) Do you mean <modernizer> in the html text or createElement("modernizer")?
>>> Those are radically different issues.
>>> The first is up to tidy, and out of our hands.
> What do we do in the first case, i.e. if we get <unknowntag>? If it's something we don't know are we putting it in the DOM but not rendering, or ignoring it?
> Also, I think *some* of those tags are actually html 5 tags i.e. <video> and <canvas> (probably the header and comment also but can't remember) and thus we
> should probably have some handling whether they're created in the html or via js (as in this case).
>
> Cheers,
> Adam.
>

--------
Kevin Carhart * 415 225 5306 * The Ten Ninety Nihilists

  parent reply	other threads:[~2016-07-02 22:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-23 11:22 Kevin Carhart
2016-06-23 16:41 ` Karl Dahlke
2016-06-27  6:39   ` Kevin Carhart
2016-07-02 16:48     ` Adam Thompson
2016-07-02 17:29       ` Karl Dahlke
2016-07-03  9:54         ` Adam Thompson
2016-07-02 22:11       ` Kevin Carhart [this message]
2016-07-03 11:12         ` Adam Thompson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.LRH.2.03.1607021453470.8153@carhart.net \
    --to=kevin@carhart.net \
    --cc=arthompson1990@gmail.com \
    --cc=edbrowse-dev@lists.the-brannons.com \
    --cc=eklhad@comcast.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).