On Monday 09 September 2002 3:56 pm, Dr. Richard E. Hawkins 
wrote:
> WHen we had the checkboxes, either the choice was persistent
> or the default was postscript.
>
> With the new scrolling box, it is necessary to click each and
> every time the window is activated, a mouseophiliac approach.
>
> hawk, mouseophobe

Interesting. We try not to update the browser unnecessarily.

Could you add a line 
        #include "support/lstrings.h"
to src/frontends/xforms/FormSendto.C and thereafter modify the 
update method (below). I suspect that it's something trivial 
like whitespace that's messing things up.

void FormSendto::update()
{
        all_formats_ = controller().allFormats();

        // Check whether the current contents of the browser will be
        // changed by loading the contents of formats
        vector<string> keys;
        keys.resize(all_formats_.size());

        vector<string>::iterator result = keys.begin();
        vector<Format const *>::const_iterator it  = 
all_formats_.begin();
        vector<Format const *>::const_iterator end = all_formats_.end();
        for (; it != end; ++it, ++result) {
-               *result = (*it)->prettyname();
+               *result = trim((*it)->prettyname());
        }

        vector<string> const browser_keys =
                getVector(dialog_->browser_formats);

        if (browser_keys == keys)
                return;

        ...
}

Angus

Reply via email to