[kmahjongg] [Bug 342159] Layout preview in settings show random layouts

[Christian Krippendorf]

https://bugs.kde.org/show_bug.cgi?id=342159

Christian Krippendorf  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #5 from Christian Krippendorf  ---
https://commits.kde.org/kmahjongg/837fffbc7e8e95a8a32ca4e3f741117ff877e352

@Albert Registered to phabricator. Thanks

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmahjongg] [Bug 342159] Layout preview in settings show random layouts

[Albert Astals Cid]

https://bugs.kde.org/show_bug.cgi?id=342159

Albert Astals Cid  changed:

   What|Removed |Added

 CC||aa...@kde.org

--- Comment #4 from Albert Astals Cid  ---
@Christian to be honest at this point i think this bug could as well be closed
without any code change, when you select random layout everything below it is
grayed out clearly indicating it no longer applies.

If you feel it makes sense to commit it, go ahead.

Also please in the future use phabricator instead of posting patches to
bugzilla, phabricator makes for easier book-keeping/reviewing/etc of patches.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmahjongg] [Bug 342159] Layout preview in settings show random layouts

[Christian Krippendorf]

https://bugs.kde.org/show_bug.cgi?id=342159

Christian Krippendorf  changed:

   What|Removed |Added

 Status|REPORTED|CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmahjongg] [Bug 342159] Layout preview in settings show random layouts

[Christian Krippendorf]

https://bugs.kde.org/show_bug.cgi?id=342159

Christian Krippendorf  changed:

   What|Removed |Added

 CC||Coding@Christian-Krippendor
   ||f.de

--- Comment #3 from Christian Krippendorf  ---
Created attachment 119076
  --> https://bugs.kde.org/attachment.cgi?id=119076=edit
Hide layout preview on random layout

-- 
You are receiving this mail because:
You are watching all bug changes.

[kmahjongg] [Bug 342159] Layout preview in settings show random layouts

[Frederik Schwarzer via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=342159

Frederik Schwarzer  changed:

   What|Removed |Added

 CC||schwar...@kde.org

--- Comment #2 from Frederik Schwarzer  ---
This problem has changed.
Here, after changing to randomised layout, the preview remains the previously
selected layout.
When changing back to another layout, the correct one is shown.

So I would not close this as fixed as long as enabling the random layout
feature keeps showing the old layout.

-- 
You are receiving this mail because:
You are watching all bug changes.