You are here: Foswiki>Tasks Web>Item775 (19 Feb 2015, CrawfordCurrie)Edit Attach
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

Binning this.

-- CrawfordCurrie - 19 Feb 2015

ItemTemplate edit

Summary Usability: Seconds are irrelevant in edit conflict
ReportedBy PeterThoeny
Codebase
SVN Range
AppliesTo Engine
Component FoswikiUIEdit
Priority Low
CurrentState No Action Required
WaitingFor
Checkins
TargetRelease major
ReleasedIn n/a
CheckinsOnBranches
trunkCheckins
masterCheckins
ItemBranchCheckins
Release01x01Checkins
Topic revision: r15 - 19 Feb 2015, CrawfordCurrie
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy