Subject: Re: FW: Commit -- Style dialog on Win32
rob.campbell@att.net
Date: Tue Jun 05 2001 - 07:58:28 CDT
I disagree. If I want to select a different style to
apply to the current paragraph (or selection), I would
use the drop down list in the formatting tool bar. The
most basic thing I would want to do with the Style
dialog is to redefine an existing style and to preview
the results in the document. The brain dead thing that
MS Word does it make "Apply" the default command, so
that when I absent-mindedly press RETURN (I use the
feature often enough to know better) after making my
changes, the style of the paragraph with the cursor
changes. If I use UNDO to undo the assignment of the
style to the paragraph (not illogical), any changes made
to the style definitions are also undone.
The "correct" operation is:
OK should be the ever present default that saves any
changed or newly created styles and closes the dialog.
If nothing changed, AW should recognize this and just
close the dialog (no undo information should be saved).
CANCEL should exit the dialog without saving any changes.
APPLY should save any changes to the style definitions,
apply the current style to the current paragraph, and
close the dialog.
A PREVIEW button should be added. This should preview
the effect on the document of the changes made to the
style definitions.
A PREVIEW APPLY button (maybe this is going too far)
should preview the effect of the current style on the
current paragraph. This is the only way to preview the
effect of a new style (or an existing style that has not
yet been used in the document).
I know that the Style dialog has preview widgets (at
least in MS Word; I haven't seen a Windows version of AW
with the Style dialog), but these are only marginally
useful. Ideally, a Preview command would shrink the
Style dialog, reformat the document, and allow the user
to browse it (multiple styles may have been changed, and
the effects may not all be visible on the current
screen). The shrunken dialog would just be a title bar
and the buttons: OK (apply style changes and close),
APPLY (apply the current style to the current paragraph
and close), and EXIT PREVIEW (return to the full size
Style dialog).
Just my thoughts.
Rob Campbell
>
> Hi Martin,
>
> The most basic thing you should be able to do with the style dialog is
> select a different already existing style from the list. We should probably
> change the button from OK to Apply. This is what Word does and it seems to
> make sense. I will probably move the logic for onApply() to XP so behaviour
> is consistent on different platforms.
>
> Bruce.
>
> > -----Original Message-----
> > From: owner-abiword-dev@abisource.com
> > [mailto:owner-abiword-dev@abisource.com]On Behalf Of Martin Sevior
> > Sent: Monday, 4 June 2001 3:56 PM
> > To: Bruce Pearson &/or Gwyn Ingham
> > Cc: Abiword-Dev
> > Subject: Re: Commit -- Style dialog on Win32
> >
> >
> >
> > Hi Bruce,
> > You Rock!
> >
> > On Mon, 4 Jun 2001, Bruce Pearson &/or Gwyn Ingham wrote:
> >
> > > Implement the Style dialog (Top) on Win32.
> > >
> > > The following is commented out on Unix. Is there a reason for this?
> > > On Win32 if I uncomment and select a style in the dialog it is
> > not set in
> > > the document.
> >
> > This code is commented out because if you delete all styles there is no
> > current style and updateDocForStyleChange(getCurrentStyle(),true) will
> > crash.
> >
> > I changed the logic in the gtk version so that this function is called if
> > you press "OK" upon exit from the "new" or "Modify" portions of the
> > dialog.
> >
> > We have to do it upon exit from the "new" or "modify" dialogs because the
> > user may wish to change several styles with the open dialog.
> >
> > Updating upon exit will only update the most recently changed dialog.
> >
> > Maybe we should remove the "cancel" and "OK" buttons. Really all the User
> > can do from the toplevel dialog is exit it.
> >
> > For the same reason maybe we should have popup window for
> > "delete" saying" "Are you really sure?"
> >
> > Once deleted the use cannot undo the change.
> >
> > Implementing undo for change sof style definitions will be really hard.
> >
> > Cheers
> >
> > Martin
> >
> >
> >
> >
> > ou can
> > either do things this way or we can discuss the corret way to handle
> > > I assume 'updateDocForStyleChange' is not working for some reason.
> > >
> > > if(m_answer == AP_Dialog_Styles::a_OK)
> > > {
> >
> > > // getView()->getCurrentBlock()->setNeedsRedraw();
> > > // getDoc()->signalListeners(PD_SIGNAL_UPDATE_LAYOUT);
> > > }
> > >
> > > CVS:
> > ----------------------------------------------------------------------
> > > CVS: Enter Log. Lines beginning with `CVS:' are removed automatically
> > > CVS:
> > > CVS: Committing in .
> > > CVS:
> > > CVS: Modified Files:
> > > CVS: src/af/xap/win/xap_Win32DialogHelper.cpp
> > > CVS: src/af/xap/win/xap_Win32DialogHelper.h
> > > CVS: src/wp/ap/win/ap_Win32Dialog_Styles.cpp
> > > CVS: src/wp/ap/win/ap_Win32Dialog_Styles.h
> > > CVS: src/wp/ap/win/ap_Win32Resources.rc2
> > > CVS: Added Files:
> > > CVS: src/wp/ap/win/ap_Win32Res_DlgStylesTop.rc2
> > > CVS:
> > ----------------------------------------------------------------------
> > >
> > >
> > >
> >
> >
>
>
This archive was generated by hypermail 2b25 : Tue Jun 05 2001 - 07:58:52 CDT