edbrowse-dev - development list for edbrowse
 help / color / mirror / Atom feed
* [Edbrowse-dev] removeAttribute
@ 2017-02-02 10:48 Kevin Carhart
  2017-02-02 11:13 ` Karl Dahlke
  0 siblings, 1 reply; 2+ messages in thread
From: Kevin Carhart @ 2017-02-02 10:48 UTC (permalink / raw)
  To: Edbrowse-dev



I might have caught a problem with removeAttribute in startwindow.

Do you think that this:
this.splice(i,1)
should say this:
this.attributes.splice(i,1)
?

Here's the code I am running in jdb to raise it.  My entries are 
preceded with > for readability.

> var blah = document.createElement("div")

> blah
[object Div]

> blah.attributes

> blah.setAttribute('a','1')
> blah.setAttribute('b','2')
> blah.setAttribute('c','3')
> blah.attributes
a,b,c

> blah.removeAttribute('b')
jdb line 673: TypeError: this.splice is not a function

> blah.removeAttribute('c')
jdb line 673: TypeError: this.splice is not a function

> blah.attributes.splice(1,1)
b

> blah.attributes
a,c




^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Edbrowse-dev]  removeAttribute
  2017-02-02 10:48 [Edbrowse-dev] removeAttribute Kevin Carhart
@ 2017-02-02 11:13 ` Karl Dahlke
  0 siblings, 0 replies; 2+ messages in thread
From: Karl Dahlke @ 2017-02-02 11:13 UTC (permalink / raw)
  To: Edbrowse-dev

> I might have caught a problem with removeAttribute in startwindow.

Well of course. It is now fixed.

Karl Dahlke

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-02-02 11:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-02 10:48 [Edbrowse-dev] removeAttribute Kevin Carhart
2017-02-02 11:13 ` Karl Dahlke

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).