[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-i18n-doc
Subject: Re: Late changes/adaptions in kpat (docbook)
From: Johannes Obermayr <johannesobermayr () gmx ! de>
Date: 2010-06-29 19:08:48
Message-ID: 201006292108.49261.johannesobermayr () gmx ! de
[Download RAW message or body]
On Tuesday 29 June 2010 20:26:44 you wrote:
> On Tue, Jun 29, 2010 at 09:10, Johannes Obermayr wrote:
> > While translating I noticed the following must be changed.
> >
> > The shuffling change is from mainwindow.cpp:210 - context: "Collect the cards in \
> > play, shuffle them and redeal them"
>
> Hello Johannes,
>
> Good catch on the shuffling discrepancy, but you chose the wrong fix.
> > ) In this particular case, the manual is correct and the action
> description is incorrect. The "Redeal" action in question is the one
> used in Grandfather. The action needs to be deterministic so that the
> solver can "see past" the next redeal, so no shuffling is done. Since
> this means only the i18n context is incorrect, I'll leave it up to you
> translation folks to decide whether this is worth changing this late
> in the release schedule.
For me it shuffles (only the ones nearest to me):
1st situation: heart 10, diamonds 6, spades queen, clubs 8, diamonds king, heart 3, \
clubs 5 2nd situation: heart 10, diamonds 5, heart 7, clubs 4, spades 10, spades 5, \
spades 8 3rd situation: heart 10, spades 8, heart ace, clubs 6, diamonds 9, clubs 9, \
spades jack
But may I am to silly following the context "shuffling" is used here. ;-)
Also end users can understand it wrong ...
>
> The "Change Appearance" change is obviously 100% correct.
>
> Of course, now that you've gotten me looking at the manual, I see a
> few other things that are no longer correct. :|
>
> Parker
>
Parker, feel free to send us necessary modifications. We will decide whether it is \
worth breaking string freeze ...
Btw. trunk should be open for new contributions/changes. Albert?
Johannes
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic