Sunday, 18 May 2014

Re: Amiga window data structures

On Sat, 17 May 2014 15:37:31 +0100, Michael Drake wrote:

> So, for example, to set the Amiga front end's scroll offset for the
> window, it starts with a struct gui_window_2 (gw2), which I assume is the
> main window containing all the tabs, then it goes to the bw (representing
> current tab?), then the gw:

Yes, that's correct. It's a bit confused as I had one structure for
everything, and then realised that wouldn't work when I added tabs.
gui_window_2 probably needs a rename to make it clearer. Most of the
frontend code uses gui_window_2 as that represents the window with the
displayed content, and prior to tabs used to be gui_window (which is
why the bw pointer is there rather than in gui_window).

> I suggest that the gw2 be changed to have a pointer to a gw representing
> the current tab, and add a pointer to bw entry in the gw. So to get the
> scroll_y it would be:
>
> gw2->gw->scroll_y
>
> And to use the core browser_window_* functions, you'd be passing the bw
> from:
>
> gw2->gw->bw
>
> Does that seem doable Chris?

Yes, that seems more logical than what we have currently.

Chris

No comments:

Post a Comment