Lars Gullik Bjønnes wrote:
Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| Hello,
|
| With this patch I am satisfying Lars desiderata. The only remaining
| issue is the naming of the TheGui class. Lars, could you please make
| up your mind about that: MainGui, QtGui, GuiInstance ?
Hmm da hmm....
class GuiImplementation : public Gui {
};
GuiImpl perhaps as a shorter name.
I'll go with GuiImplementation, thanks.
| @@ -583,7 +583,7 @@
| // scrollDocView(new_top_y);
| //
| // // Update the scrollbar.
| -// workarea().setScrollbarParams(t->height(), top_y(), defaultRowHeight());
| +// work_area_->.setScrollbarParams(t->height(), top_y(),
defaultRowHeight());
| }
The commented code got mangeld.
I plan to remove this workarea() access method so it's better to not
leave any trace.
Abdel.
