search

keeleysam / tenfourfox

Automatically exported from code.google.com/p/tenfourfox
0 stars 0 forks source link

double-click words requires hacking nsTextFrameThebes #122

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Double-click to select words in 10 requires hacking 
nsTextFrameThebes.cpp/nsTextFrame::PeekOffsetWord to make isWordBreakBefore == 
isWhitespace. This seems to work okay for English and Japanese or at least 
similar enough to previous behaviour, but this shouldn't be necessary. What 
broke?

Original issue reported on code.google.com by classi...@floodgap.com on 8 Jan 2012 at 6:23

GoogleCodeExporter commented 9 years ago 
If there's anything wrong with text fragments or the like it will almost surely 
be altivec accelerated text conversion...

Original comment by Tobias.N...@gmail.com on 8 Jan 2012 at 8:58

GoogleCodeExporter commented 9 years ago 
I had thought of that, but I didn't have a G3 build up. This is a one line 
patch, so when I make the G3 beta later today I will comment the line out and 
see if it still happens in the G3 build.

Original comment by classi...@floodgap.com on 8 Jan 2012 at 3:58

GoogleCodeExporter commented 9 years ago 
You're off the hook -- it happens in the G3 build too. So I'm baffled.

Original comment by classi...@floodgap.com on 8 Jan 2012 at 7:30

GoogleCodeExporter commented 9 years ago 
This is related to issue 129, although the fix doesn't fix issue 129 itself. 
Gonna dupe this into that.

Original comment by classi...@floodgap.com on 5 Feb 2012 at 12:31