Menu

#90 Extra train buttons on review page

open
5
2004-05-24
2004-03-16
Tony Meyer
No

From personal email:

"""
I've finally installed the latest SpamBayes version,
and the change of ordering is a big help. However, one
more suggestion: there should be a "train" button under
all three sections, not just at the bottom. Thus it's
not necessary to scroll all the way down after looking
at just the unsure or ham messages.
"""

I agree with this, so if someone wants to get around to
adding the extra buttons, that would be cool :) I'm
not sure if it's just adding the extra html, since
maybe the dynamic creation of the page will throw
things off (not sure at all).

Discussion

  • Richie Hindle

    Richie Hindle - 2004-03-17

    Logged In: YES
    user_id=85414

    Adding copies of the Train button would work, but
    FWIW I don't think it's a good idea. It's not obvious
    whether each button would submit the whole page,
    or just the section it was immediately underneath,
    or all the sections above it. Maybe changing the text
    could help, but if it says "Train all" or something then
    than only makes you wonder what the alternative is...
    then you end up adding "Train this section" buttons
    and the whole thing becomes overengineered.

     
  • Richie Hindle

    Richie Hindle - 2004-03-17
    • summary: Extra "Train" buttons on review page. --> Extra
     
  • Tony Meyer

    Tony Meyer - 2004-03-17
    • status: open --> closed
     
  • Tony Meyer

    Tony Meyer - 2004-03-17

    Logged In: YES
    user_id=552329

    Good point! I hadn't thought of that.

    There hasn't been a clamouring for this anyway, so I'll close
    it, and refer anyone that asks to the closed tracker :)

     
  • Tony Meyer

    Tony Meyer - 2004-03-17
    • assigned_to: nobody --> anadelonbrin
    • summary: Extra --> Extra train buttons on review page
     
  • Tony Meyer

    Tony Meyer - 2004-05-24

    Logged In: YES
    user_id=552329

    See also:

    [ 955892 ] Train button should be on top
    <http://sourceforge.net/tracker/?group_id=61702&atid=498106&func=detail&aid=955892>

     
  • Tony Meyer

    Tony Meyer - 2004-05-24

    Logged In: YES
    user_id=552329

    Reopening, at least temporarily.

    Richie (if you're about), what about just one more "train"
    button, at the top of the page (above any messages)? This
    is requested every now and then, because lots of people just
    do the unsures and trust whatever default setting they have
    for ham/spam, so don't want to scroll down.

     
  • Tony Meyer

    Tony Meyer - 2004-05-24
    • status: closed --> open
     
  • Richie Hindle

    Richie Hindle - 2004-05-26

    Patch to add a Train button at the top of the list.

     
  • Richie Hindle

    Richie Hindle - 2004-05-26

    Logged In: YES
    user_id=85414

    I think it's a bad idea to encourage people to train
    without even looking at what they're training on.
    Mistakes will get reinforced without people's knowledge.
    But maybe the customer is always right; I don't know.

    Attached is a patch to add another train button at the
    top of the list. I don't want to commit it because I
    think it's bad UI design... Tony, you have a much better
    idea than I do about how people use the UI, so I'll leave
    it up to you (and anyone else with an opinion 8-) whether
    you commit it or not.

    copping-out-passing-the-buck-call-it-what-you-will-ly yrs,
    Richie

     
  • Tony Meyer

    Tony Meyer - 2004-05-31

    Logged In: YES
    user_id=552329

    I share your opinion about encouraging people to blindly
    train, and also the uncertainty about whether to just
    believe the customer is always right.

    There are lots of fancy things we could do so that it was
    only there if they enable an option, or after they've
    trained x messages or something, but those all feel pretty
    kludgy.

    I'm going to copy your excellent example of passing the buck
    and leave this open with the patch. For the moment, anyone
    that cares can run from source with the patch. If there are
    more requests for this over the next few months, then maybe
    I'll apply it.

     

Log in to post a comment.