• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

I just got this message on an edit conflict:

They have been editing for 4 minutes 38 seconds and their lease is still active for another 55 minutes 22 seconds.

Nobody cares about number of seconds to wait. I suggest to change that to:

They have been editing for 4 minutes and their lease is still active for another 55 minutes.

-- PTh

aaaaaargh - i don't care about them being there - but i sure don't want to be told they have been editing for 0 minutes etc...

-- SD

How about:

$time = 60 if( $time < 60); # unit seconds

-- PTh

Some might care that the count down does not decrease when they try again if we hide seconds.

If you think it is wordy, try shortening it to '4min 55s'

Frankly, given everything else, I don't think this conv. is worth having. I suggest you discard it.

-- MC


Good idea to display in minutes unless the time is less than 1 minute and seconds are displayed.

Sure there will be some people that don't see the time decrease, but I find this a good solution for all other 99.99%.

AC

righto - i'm tossing this on the enhancement pile - too pointless for me to look into - i've enough of a hard time concentrating on the real bugs while i'm sick -- SD

Undeferred, post Dakar CC

Waste of time. Discarded. CC


I don't agree this is a waste of time, but it is a detail. If there was a label Low Enhancement I would use that. Using Low now.

In Edit.pm:

                    # The lease is active
                    $def = 'lease_active';
                    $past = TWiki::Time::formatDelta(
                        $t - $lease->{taken}, $session->{i18n} );
                    $future = TWiki::Time::formatDelta(
                        $lease->{expires} - $t, $session->{i18n} );
Should become something like this:
                    # The lease is active
                    $def = 'lease_active';
                    my $pastseconds = $t - $lease->{taken};
                    if ($pastseconds > 60) $pastseconds %= 60;
                    my $futureseconds = $lease->{expires} - $t;
                    if ($futureseconds > 60) $futureseconds %= 60;
                    $past = TWiki::Time::formatDelta(
                        $pastseconds, $session->{i18n} );
                    $future = TWiki::Time::formatDelta(
                        futureseconds, $session->{i18n} );
but I am afraid that formatDelta will just display 0 seconds.

So, add a parameter to formatDelta, like round ?

AC

ItemTemplate
Summary Usability: Seconds are irrelevant in edit conflict
ReportedBy PeterThoeny
SVN Range

AppliesTo Engine
Component

Priority Low
CurrentState New
WaitingFor

TargetRelease major
Edit | Attach | Watch | Print version | History: r12 < r11 < r10 < r9 < r8 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r12 - 2006-02-18 - ArthurClemens
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback