[chromium-dev] Re: ScrollBar Implementation
There are different kinds of scrollbar supported by views... that native scrollbar and the bitmap scrollbar. bitmap scrollbar isn't used right now but likely will be again. ScrollBar is the base class of both NativeScrollbar and Bitmapscrollbar. Scrollview has a ScrollBar* so it doesn't have to care about the particular implementation. -Ben On Tue, Mar 24, 2009 at 6:36 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi chromium, I cc'd Ben cause he seems to be working on UI on chrome alot :x Sorry Ben! I am working on a feature http://code.google.com/p/chromium/issues/detail?id=7231 which requires context menu support for Chromium scroll bars. While searching the code I have come up with these classes: - NativeScrollBar - ScrollBar - ScrollView - BitmapScrollBar The context menu functionality should have the following (system wide in windows, every window that has scrollbar, has that context menu) Horizontal: 1) Scroll Here 2) Left Edge 3) Right Edge 4) Page Left 5) Page Right 6) Scroll Left 7) Scroll Right Vertical: 1) Scroll Here 2) Top 3) Bottom 4) Page Up 5) Page Down 6) Scroll Up 7) Scroll Down So my question is as follows, Chromium uses ScrollView as its main scroll bar. It doesn't use NativeScrollBar, nor BitmapScrollBar. So why do those exist? Why do we have a ScrollBar class? Why can't we just use ScrollView and instead of passing in a ScrollBar, are we assuming Chromium will have *many* different types of scrollbars? Why are we not using native windows scrollbar support, since chromium is creating different views in every operating system, why not just use the standard windows components? There are many functionality within the scrollbar, how would you guys advice me to continue? Should I create a separate class called ScrollBarContextMenu and implement the horizontal and vertical context menu for scroll bars? There must be an easier way, to just use windows/gtk/mac controls for scrollbar support, instead of recreating it from scratch. Ideas? --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: ScrollBar Implementation
Do you recommend me to refactor the Context Menu code from BitmapScrollBar into ScrollBar so that both will have context menu implementations? Currently, the context menu is the page context menu that has reload, print, view page info, etc. Would that be the best approach fixing this feature bug. http://code.google.com/p/chromium/issues/detail?id=7231 -m0 On Wed, Mar 25, 2009 at 3:49 PM, Ben Goodger (Google) b...@chromium.org wrote: There are different kinds of scrollbar supported by views... that native scrollbar and the bitmap scrollbar. bitmap scrollbar isn't used right now but likely will be again. ScrollBar is the base class of both NativeScrollbar and Bitmapscrollbar. Scrollview has a ScrollBar* so it doesn't have to care about the particular implementation. -Ben On Tue, Mar 24, 2009 at 6:36 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi chromium, I cc'd Ben cause he seems to be working on UI on chrome alot :x Sorry Ben! I am working on a feature http://code.google.com/p/chromium/issues/detail?id=7231 which requires context menu support for Chromium scroll bars. While searching the code I have come up with these classes: - NativeScrollBar - ScrollBar - ScrollView - BitmapScrollBar The context menu functionality should have the following (system wide in windows, every window that has scrollbar, has that context menu) Horizontal: 1) Scroll Here 2) Left Edge 3) Right Edge 4) Page Left 5) Page Right 6) Scroll Left 7) Scroll Right Vertical: 1) Scroll Here 2) Top 3) Bottom 4) Page Up 5) Page Down 6) Scroll Up 7) Scroll Down So my question is as follows, Chromium uses ScrollView as its main scroll bar. It doesn't use NativeScrollBar, nor BitmapScrollBar. So why do those exist? Why do we have a ScrollBar class? Why can't we just use ScrollView and instead of passing in a ScrollBar, are we assuming Chromium will have *many* different types of scrollbars? Why are we not using native windows scrollbar support, since chromium is creating different views in every operating system, why not just use the standard windows components? There are many functionality within the scrollbar, how would you guys advice me to continue? Should I create a separate class called ScrollBarContextMenu and implement the horizontal and vertical context menu for scroll bars? There must be an easier way, to just use windows/gtk/mac controls for scrollbar support, instead of recreating it from scratch. Ideas? --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: ScrollBar Implementation
No. What's not working? -Ben On Wed, Mar 25, 2009 at 12:54 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Do you recommend me to refactor the Context Menu code from BitmapScrollBar into ScrollBar so that both will have context menu implementations? Currently, the context menu is the page context menu that has reload, print, view page info, etc. Would that be the best approach fixing this feature bug. http://code.google.com/p/chromium/issues/detail?id=7231 -m0 On Wed, Mar 25, 2009 at 3:49 PM, Ben Goodger (Google) b...@chromium.org wrote: There are different kinds of scrollbar supported by views... that native scrollbar and the bitmap scrollbar. bitmap scrollbar isn't used right now but likely will be again. ScrollBar is the base class of both NativeScrollbar and Bitmapscrollbar. Scrollview has a ScrollBar* so it doesn't have to care about the particular implementation. -Ben On Tue, Mar 24, 2009 at 6:36 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi chromium, I cc'd Ben cause he seems to be working on UI on chrome alot :x Sorry Ben! I am working on a feature http://code.google.com/p/chromium/issues/detail?id=7231 which requires context menu support for Chromium scroll bars. While searching the code I have come up with these classes: - NativeScrollBar - ScrollBar - ScrollView - BitmapScrollBar The context menu functionality should have the following (system wide in windows, every window that has scrollbar, has that context menu) Horizontal: 1) Scroll Here 2) Left Edge 3) Right Edge 4) Page Left 5) Page Right 6) Scroll Left 7) Scroll Right Vertical: 1) Scroll Here 2) Top 3) Bottom 4) Page Up 5) Page Down 6) Scroll Up 7) Scroll Down So my question is as follows, Chromium uses ScrollView as its main scroll bar. It doesn't use NativeScrollBar, nor BitmapScrollBar. So why do those exist? Why do we have a ScrollBar class? Why can't we just use ScrollView and instead of passing in a ScrollBar, are we assuming Chromium will have *many* different types of scrollbars? Why are we not using native windows scrollbar support, since chromium is creating different views in every operating system, why not just use the standard windows components? There are many functionality within the scrollbar, how would you guys advice me to continue? Should I create a separate class called ScrollBarContextMenu and implement the horizontal and vertical context menu for scroll bars? There must be an easier way, to just use windows/gtk/mac controls for scrollbar support, instead of recreating it from scratch. Ideas? --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: ScrollBar Implementation
There is no context menu for the ScrollBar. -m0 On Wed, Mar 25, 2009 at 3:55 PM, Ben Goodger (Google) b...@chromium.org wrote: No. What's not working? -Ben On Wed, Mar 25, 2009 at 12:54 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Do you recommend me to refactor the Context Menu code from BitmapScrollBar into ScrollBar so that both will have context menu implementations? Currently, the context menu is the page context menu that has reload, print, view page info, etc. Would that be the best approach fixing this feature bug. http://code.google.com/p/chromium/issues/detail?id=7231 -m0 On Wed, Mar 25, 2009 at 3:49 PM, Ben Goodger (Google) b...@chromium.org wrote: There are different kinds of scrollbar supported by views... that native scrollbar and the bitmap scrollbar. bitmap scrollbar isn't used right now but likely will be again. ScrollBar is the base class of both NativeScrollbar and Bitmapscrollbar. Scrollview has a ScrollBar* so it doesn't have to care about the particular implementation. -Ben On Tue, Mar 24, 2009 at 6:36 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi chromium, I cc'd Ben cause he seems to be working on UI on chrome alot :x Sorry Ben! I am working on a feature http://code.google.com/p/chromium/issues/detail?id=7231 which requires context menu support for Chromium scroll bars. While searching the code I have come up with these classes: - NativeScrollBar - ScrollBar - ScrollView - BitmapScrollBar The context menu functionality should have the following (system wide in windows, every window that has scrollbar, has that context menu) Horizontal: 1) Scroll Here 2) Left Edge 3) Right Edge 4) Page Left 5) Page Right 6) Scroll Left 7) Scroll Right Vertical: 1) Scroll Here 2) Top 3) Bottom 4) Page Up 5) Page Down 6) Scroll Up 7) Scroll Down So my question is as follows, Chromium uses ScrollView as its main scroll bar. It doesn't use NativeScrollBar, nor BitmapScrollBar. So why do those exist? Why do we have a ScrollBar class? Why can't we just use ScrollView and instead of passing in a ScrollBar, are we assuming Chromium will have *many* different types of scrollbars? Why are we not using native windows scrollbar support, since chromium is creating different views in every operating system, why not just use the standard windows components? There are many functionality within the scrollbar, how would you guys advice me to continue? Should I create a separate class called ScrollBarContextMenu and implement the horizontal and vertical context menu for scroll bars? There must be an easier way, to just use windows/gtk/mac controls for scrollbar support, instead of recreating it from scratch. Ideas? --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: ScrollBar Implementation
As Peter said, the ScrollBar object is not responsible for drawing scroll bars on pages, which is what the bug is about. Brett On Wed, Mar 25, 2009 at 1:15 PM, Mohamed Mansour m0.interact...@gmail.com wrote: There is no context menu for the ScrollBar. -m0 On Wed, Mar 25, 2009 at 3:55 PM, Ben Goodger (Google) b...@chromium.org wrote: No. What's not working? -Ben On Wed, Mar 25, 2009 at 12:54 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Do you recommend me to refactor the Context Menu code from BitmapScrollBar into ScrollBar so that both will have context menu implementations? Currently, the context menu is the page context menu that has reload, print, view page info, etc. Would that be the best approach fixing this feature bug. http://code.google.com/p/chromium/issues/detail?id=7231 -m0 On Wed, Mar 25, 2009 at 3:49 PM, Ben Goodger (Google) b...@chromium.org wrote: There are different kinds of scrollbar supported by views... that native scrollbar and the bitmap scrollbar. bitmap scrollbar isn't used right now but likely will be again. ScrollBar is the base class of both NativeScrollbar and Bitmapscrollbar. Scrollview has a ScrollBar* so it doesn't have to care about the particular implementation. -Ben On Tue, Mar 24, 2009 at 6:36 PM, Mohamed Mansour m0.interact...@gmail.com wrote: Hi chromium, I cc'd Ben cause he seems to be working on UI on chrome alot :x Sorry Ben! I am working on a feature http://code.google.com/p/chromium/issues/detail?id=7231 which requires context menu support for Chromium scroll bars. While searching the code I have come up with these classes: - NativeScrollBar - ScrollBar - ScrollView - BitmapScrollBar The context menu functionality should have the following (system wide in windows, every window that has scrollbar, has that context menu) Horizontal: 1) Scroll Here 2) Left Edge 3) Right Edge 4) Page Left 5) Page Right 6) Scroll Left 7) Scroll Right Vertical: 1) Scroll Here 2) Top 3) Bottom 4) Page Up 5) Page Down 6) Scroll Up 7) Scroll Down So my question is as follows, Chromium uses ScrollView as its main scroll bar. It doesn't use NativeScrollBar, nor BitmapScrollBar. So why do those exist? Why do we have a ScrollBar class? Why can't we just use ScrollView and instead of passing in a ScrollBar, are we assuming Chromium will have *many* different types of scrollbars? Why are we not using native windows scrollbar support, since chromium is creating different views in every operating system, why not just use the standard windows components? There are many functionality within the scrollbar, how would you guys advice me to continue? Should I create a separate class called ScrollBarContextMenu and implement the horizontal and vertical context menu for scroll bars? There must be an easier way, to just use windows/gtk/mac controls for scrollbar support, instead of recreating it from scratch. Ideas? --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---
[chromium-dev] Re: ScrollBar Implementation
Without looking at the code, some thoughts: * Scrollbars in web content differ from scrollbars in native content. Native content can use native scrollbars. Web content cannot (web content cannot use native widgets within pages as then pages can just recursively include 10,000 iframes and take down all of GDI and DOS your whole system). So at least some of the distinction comes from this. * I suggest you determine where each of the existing classes are used, and how other code in those areas shows context menus, since I suspect that context menus work very differently in web content versus native controls. PK --~--~-~--~~~---~--~~ Chromium Developers mailing list: chromium-dev@googlegroups.com View archives, change email options, or unsubscribe: http://groups.google.com/group/chromium-dev -~--~~~~--~~--~--~---