edbrowse-dev - development list for edbrowse
 help / color / mirror / Atom feed
From: Karl Dahlke <eklhad@comcast.net>
To: edbrowse-dev@lists.the-brannons.com
Subject: [edbrowse-dev] dataset / offsetWidth
Date: Sun, 16 Jun 2019 14:44:55 -0400	[thread overview]
Message-ID: <20190516144455.eklhad@comcast.net> (raw)
In-Reply-To: <alpine.DEB.2.21.1906152356310.29748@carhart.net>

[-- Attachment #1: Type: text/plain, Size: 1198 bytes --]

if (strncmp(a[i],"data-",5)==0) {
jsobjtype dso = get_property_object(t->jv,"dataset");
if(dso)
set_property_string(dso, a[i]+5, u);
}

That looks ok, do you want me to code or do you want to code and make a pull request.

I don't mind setting default heights and widths on just about everything, I guess we have to,
and it's *way* too much bother to figure out what they should really be, cause we'd have to essentially lay out the screen, and edbrowse doesn't have a screen.
But 1024 by 768 is prety heavy handed.
A span with 2 letters shouldn't take the whole screen.
I would make the height 16 the pixels of a single character, and the width like 120 or about 15 characters.
Some day when we're ambitious we can guess how much "stuff" is in that element and if it's 50 characters then make it wider,
but we have to start somewhere.
Yes you would takc these onto c.prototype.
Again do you want to or should I/

c.prototype.clientHeight = 16;
c.prototype.clientWidth = 120;
c.prototype.offsetHeight = 16;
c.prototype.offsetWidth = 120;
c.prototype.scrollHeight = 16;
c.prototype.scrollWidth = 120;
c.prototype.scrollTop = 0;
c.prototype.scrollLeft = 0;


Karl Dahlke

  reply	other threads:[~2019-06-16 18:44 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-16 16:24 Kevin Carhart
2019-06-16 18:44 ` Karl Dahlke [this message]
2019-06-16 19:21   ` Kevin Carhart

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=20190516144455.eklhad@comcast.net \
    --to=eklhad@comcast.net \
    --cc=edbrowse-dev@lists.the-brannons.com \
    /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).