Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
In-Reply-To: <20200205145223.66869-1-larry@theclapp.org>
Date: Thu, 06 Feb 2020 08:17:28 +0100
Cc: "Larry Clapp" <larry@theclapp.org>
Subject: Re: [PATCH] widget: add some utility methods on Editor
From: "Elias Naur" <mail@eliasnaur.com>
To: "Larry Clapp" <larry@theclapp.org>, <~eliasnaur/gio-patches@lists.sr.ht>
Message-Id: <C0EWBBWT3Q14.S4IMFZ8HJTGL@testmac>

On Wed Feb 5, 2020 at 9:52 AM, Larry Clapp wrote:
> - Focused: returns whether editor is focused
> - CaretPos: returns the text line & column number, and the x & y pixel
> coordinates of the caret.
>
>=20
> Note: Returns them in that order. Which seems odd: line & column is
> basically "y, x", but then the pixel coordinates are x & y. So
> wondering if swapping line & column in CaretPos (and possibly in
> Editor.layoutCaret which it calls) would be appropriate?

Other than having many return values, how often is both the text
position (line, column) and absolute position (x, y) needed?
It seems to me CaretPos should be split into CaretPos (line, column) and
CaretCoords (x, y). Better names welcome.

> - NumLines: returns the number of text lines in the editor
>