kdegraphics/kghostview

[Waldo Bastian]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Sunday 06 July 2003 21:43, Nadeem Hasan wrote:
> WARNING: Unsanitized content follows.
>
> On Sun July 6 2003 03:22 pm, you wrote:
> > WARNING: Unsanitized content follows.
> >
> > On Sunday 06 July 2003 20:55, Nadeem Hasan wrote:
> > > CVS commit by nhasan:
> > >
> > > * Move "Full Screen" option to view menu and disable it in the initial
> > > state.
> >
> > It was moved to Settings as part of the fix for
> > http://bugs.kde.org/show_bug.cgi?id=57935
>
> I won't call it a fix. That option does not belong in Settings menu. Don't
> you agree?

It is a function that influences the general layout of the application and as 
such not so different from Show Toolbar/MenuBar/StatusBar. Those are placed 
in Settings although you can argue if that really makes sense.

The purpose of the View menu is to provide options that influence the 
presentation of information within the main-window. I don't think "Full 
Screen" in general fits in with that. For applications such as kghostview and 
kview the View menu might not be a bad choice since within the context of 
those applications the action can be seen as a image-scaling function. But I 
don't think having it in the View menu would make sense for other 
applications.

So, although not perfect, I think that the settings menu is the best location 
to standardize for the "Full Screen" option.

Cheers,
Waldo
- -- 
bastian at kde.org -=|[ SuSE, The Linux Desktop Experts ]|=- bastian at suse.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQE/CUF0N4pvrENfboIRAuu2AJ93tc2R7MaEbTaojkKxXuQeCFhsUgCeMuK9
HCe7QU4fDUXBe8FN8a7hQfY=
=BTw4
-----END PGP SIGNATURE-----