- Dillo-dev - lists.mailman3.com

Window close dialog

by jcid＠dillo.org

On Mon, Sep 29, 2008 at 04:03:55PM +0200, Johannes Hofmann wrote: > Hi, > > On Mon, Sep 29, 2008 at 09:48:58AM -0400, Jorge Arellano Cid wrote: > > Hi, > > > > Can somebody please make the dialog for closing a window > > with tabs open? > > "Close Window" only closes the current tab anyway. Yes. > "Exit Dillo" closes all windows including tabs. > So, do you think about a dialog in the "Exit Dillo" case when > there is more than one tab or more than one window or both? Both, plus the window manager's "close window" if there're open Tabs. > Do we have the number of open windows somewhere? In bw.cc's num_bws. -- Cheers Jorge.-

Sept. 29, 2008

what crashes remain?

by jcid＠dillo.org

On Mon, Sep 29, 2008 at 02:09:36PM +0000, corvid wrote: > Jorge wrote: > > On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > > > all segfault. I thought somebody said something (that I didn't pay > > > attention to) about the file menu since tabs were added, but if so, > > > I can't find that mail. > > > > Please check the patch in CVS. > > > > FLTK2 has a bug in that it regards the first menu item letter as > > a shortcut. > > > > Note: I tried with invisible shortcuts doing nothing, > > removing shortcuts and all that stuff, until I found this way > > to workaround the problem. > > > > Note2: Alt-W and Alt-X are accelerators, so they're OK. > > > > AFAIS there's no more segfault here. > > Alt-W and Alt-X still segfault for me. Hmm... What fltk version are you using? Anyway, as a workaround, try removing the ampersand in the labels for "Close Window" and "Exit Dillo", in UI::make_menubar(). > What's an accelerator? Kind of a shortcut that you set with an ampersand in the label of an fltk::Item. Disclaimer: "accelerator" may not be the correct name for it. -- Cheers Jorge.-

Sept. 29, 2008

what crashes remain?

by corvid＠lavabit.com

Jorge wrote: > On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > > all segfault. I thought somebody said something (that I didn't pay > > attention to) about the file menu since tabs were added, but if so, > > I can't find that mail. > > Please check the patch in CVS. > > FLTK2 has a bug in that it regards the first menu item letter as > a shortcut. > > Note: I tried with invisible shortcuts doing nothing, > removing shortcuts and all that stuff, until I found this way > to workaround the problem. > > Note2: Alt-W and Alt-X are accelerators, so they're OK. > > AFAIS there's no more segfault here. Alt-W and Alt-X still segfault for me. What's an accelerator?

Sept. 29, 2008

Window close dialog

by Johannes.Hofmann＠gmx.de

Hi, On Mon, Sep 29, 2008 at 09:48:58AM -0400, Jorge Arellano Cid wrote: > Hi, > > Can somebody please make the dialog for closing a window > with tabs open? "Close Window" only closes the current tab anyway. "Exit Dillo" closes all windows including tabs. So, do you think about a dialog in the "Exit Dillo" case when there is more than one tab or more than one window or both? Do we have the number of open windows somewhere? Cheers, Johannes

Sept. 29, 2008

Window close dialog

by jcid＠dillo.org

Hi, Can somebody please make the dialog for closing a window with tabs open? -- Cheers Jorge.-

Sept. 29, 2008

what crashes remain?

by jcid＠dillo.org

On Sun, Sep 28, 2008 at 12:52:47AM +0000, corvid wrote: > Johannes wrote: > > On Sat, Sep 27, 2008 at 05:29:28PM +0000, corvid wrote: > > > I can't think of any other crashes, but I don't remember whether > > > Johannes still gets bus errors. > > > > No, they got fixed with > > http://lists.auriga.wearlab.de/pipermail/dillo-dev/2008-July/004649.html > > > > The only issues I'm seeing are the 128 tabs limit from fltk and the > > redraw storms related to the MenuTabPager. > > Playing with the file menu, I see that Alt-W, Alt-E, Alt-X, Alt-C > all segfault. I thought somebody said something (that I didn't pay > attention to) about the file menu since tabs were added, but if so, > I can't find that mail. Please check the patch in CVS. FLTK2 has a bug in that it regards the first menu item letter as a shortcut. Note: I tried with invisible shortcuts doing nothing, removing shortcuts and all that stuff, until I found this way to workaround the problem. Note2: Alt-W and Alt-X are accelerators, so they're OK. AFAIS there's no more segfault here. -- Cheers Jorge.-

Sept. 29, 2008

patch: show_msg=NO disable debug messages

by onepoint＠starurchin.org

On Thu, Sep 25, 2008 at 02:03:39PM -0400, Jorge Arellano Cid wrote: > In other words, for me, its OK to get rid of DEBUG_MSG unless > somebody else finds good use for it. I usually add my own debugging statements rather than use DEBUG_MSG . Maybe that says more about my ego than anything else :-) but FWIW I wouldn't miss DEBUG_MSG either. Regards, Jeremy Henty

Sept. 29, 2008

patch: changeLinkColor

by jcid＠dillo.org

On Sun, Sep 28, 2008 at 04:40:53PM -0400, Jorge Arellano Cid wrote: > On Sun, Sep 28, 2008 at 09:20:17PM +0200, Johannes Hofmann wrote: > > On Sun, Sep 28, 2008 at 02:54:56PM -0400, Jorge Arellano Cid wrote: > > > On Sun, Sep 28, 2008 at 05:55:10PM +0000, corvid wrote: > > > > Encouraged by the idea that I am going in the right direction, I have > > > > made a new one with spaceStyle: > > > > > > Funny how we were working on the same patch a the same time! > > > > > > I had an ancient version, and gave it a review. > > > The link search is linear inside textblock. A linux gazette page > > > showed more than 33000 words to color a link. On bigger pages it may be worst. > > > It depends on the structure. For instance our bookmarks uses tables > > > and the link is usually found in less than 10 words. > > > > > > The patch in CVS reduce the 35000 steps to 1500 or so (the > > > sorted link list is sparse, so there's a penalty for looking > > > for an actual link to compare to). > > > > > > > Not sure I understand this optimization. > > It is assuming that link numbers are always increasing, right? > > Right. > > > Can we assume this in dw/*? > > I checked html.cc and x_link is crescent. Not sure about the rest. > > > src/html.cc seems to be implemented currently in way that this is > > true. But perhaps we could/should reuse link numbers for equal > > destination urls in the future? > > In that case it'd need a revision. > > > Or someone might just choose hashes of the url string as link > > number. > > > > BTW. with both implementations I now see issues with this example: > > > > <html> > > <body> > > <a href="http://www.dillo.org">dillo</a> > > <a href="http://slashdot.org">slashdot</a> > > <a href="http://www.dillo.org">dillo</a> > > </body> > > </html> > > > > When opening the first link in a new window, it doesn't change color > > in the original one. > > When opening it in a tab, the first one changes color. After a > > reload also the second one changes color. > > Am I missing something? > > Yes, the link color change is meant as a visual cue for the user > to know the click is being processed. We could leave the link alone > and change the mouse cursor instead. > > The link not changing color when openning it with the menu > is IMO a bug, I'm looking for... It seems that carrying around widget, link and html->visited_color could do the trick. Maybe it's not worth the effort as the new window is enough visual feedback, and also the new tab... Comments? -- Cheers Jorge.-

Sept. 29, 2008

patch: changeLinkColor

by jcid＠dillo.org

On Sun, Sep 28, 2008 at 09:20:17PM +0200, Johannes Hofmann wrote: > On Sun, Sep 28, 2008 at 02:54:56PM -0400, Jorge Arellano Cid wrote: > > On Sun, Sep 28, 2008 at 05:55:10PM +0000, corvid wrote: > > > Encouraged by the idea that I am going in the right direction, I have > > > made a new one with spaceStyle: > > > > Funny how we were working on the same patch a the same time! > > > > I had an ancient version, and gave it a review. > > The link search is linear inside textblock. A linux gazette page > > showed more than 33000 words to color a link. On bigger pages it may be worst. > > It depends on the structure. For instance our bookmarks uses tables > > and the link is usually found in less than 10 words. > > > > The patch in CVS reduce the 35000 steps to 1500 or so (the > > sorted link list is sparse, so there's a penalty for looking > > for an actual link to compare to). > > > > Not sure I understand this optimization. > It is assuming that link numbers are always increasing, right? Right. > Can we assume this in dw/*? I checked html.cc and x_link is crescent. Not sure about the rest. > src/html.cc seems to be implemented currently in way that this is > true. But perhaps we could/should reuse link numbers for equal > destination urls in the future? In that case it'd need a revision. > Or someone might just choose hashes of the url string as link > number. > > BTW. with both implementations I now see issues with this example: > > <html> > <body> > <a href="http://www.dillo.org">dillo</a> > <a href="http://slashdot.org">slashdot</a> > <a href="http://www.dillo.org">dillo</a> > </body> > </html> > > When opening the first link in a new window, it doesn't change color > in the original one. > When opening it in a tab, the first one changes color. After a > reload also the second one changes color. > Am I missing something? Yes, the link color change is meant as a visual cue for the user to know the click is being processed. We could leave the link alone and change the mouse cursor instead. The link not changing color when openning it with the menu is IMO a bug, I'm looking for... BTW, the linear search patch is much simpler and as link clicking is not a performance critical task it could be left implemented in O(n). -- Cheers Jorge.-

Sept. 29, 2008

patch: changeLinkColor

by corvid＠lavabit.com

Johannes wrote: > BTW. with both implementations I now see issues with this example: > > <html> > <body> > <a href="http://www.dillo.org">dillo</a> > <a href="http://slashdot.org">slashdot</a> > <a href="http://www.dillo.org">dillo</a> > </body> > </html> > > When opening the first link in a new window, it doesn't change color > in the original one. > When opening it in a tab, the first one changes color. After a > reload also the second one changes color. > Am I missing something? Middle click changes the color for me whether it opens a tab or a window.

Sept. 29, 2008