Bug 35319 - unable to click bottom right corner
Summary: unable to click bottom right corner
Status: RESOLVED FIXED
Alias: None
Product: knavalbattle
Classification: Applications
Component: general (show other bugs)
Version: 1.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Nikolas Zimmermann
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-11-24 19:48 UTC by Graf Zahl
Modified: 2003-06-13 06:05 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Graf Zahl 2001-11-24 19:47:21 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           kbattleship
Version:           1.0 (using KDE 2.2.1 )
Severity:          normal
Installed from:    SuSE RPMs
Compiler:          gcc version 2.95.3 20010315 (SuSE)
OS:                Linux
OS/Compiler notes: kbattleship came precompiled with SuSE distribution

When playing kbattleship (single or multiplayer) I *sometimes* can't click (i.e. shoot) the bottom right corner of the enemy's battlefield. I wasn't able to make out when it works and when it doesn't. It seems to work if the very first shot in a game is placed in the bottom right. If it's the second or later shot it works sometimes other times nothing at all happens or the "shot" appears in the bottom left or top right corner.

(Submitted via bugs.kde.org)
Comment 1 Jim Bob 2003-04-05 02:39:20 UTC
I have a similar problem with the KBattleship included in Debian/KDE 3.1
(packaged by kde.org) on both x86 and PowerPC. Sometimes clicks aren't
acknowledged (ie, no shot is made). As far as I can tell, battleship always
works properly if the pointer stays within the grid as you move it to its
destination (ie the place where you shoot). If the pointer moves from outside
the grid and directly onto the destination, then it might not work. The cell in
the lower-right corner is the most onerous.
Comment 2 Maksim Orlovich 2003-06-13 06:05:55 UTC
Subject: kdegames/kbattleship/kbattleship

CVS commit by orlovich: 

Fix nonsense field geometry calculations that worked by a miracle and caused the
end of the last row and column to be excluded from event handling (#35319)
CCMAIL:35319-done@bugs.kde.org


  M +2 -2      kbattlefield.cpp   1.52


--- kdegames/kbattleship/kbattleship/kbattlefield.cpp  #1.51:1.52
@@ -227,9 +227,9 @@ int KBattleField::rectX()
 QRect KBattleField::ownRect()
 {
-        return QRect(ownXPosition(), ownYPosition(), ownXPosition() + ((m_ownfieldx - 1) * gridSize()), ownYPosition() + ((m_ownfieldy - 1) * gridSize()));
+        return QRect(ownXPosition(), ownYPosition(), m_ownfieldx * gridSize(), m_ownfieldy * gridSize());
 }
 
 QRect KBattleField::enemyRect()
 {
-        return QRect(enemyXPosition(), enemyYPosition(), rectX() + ((m_enemyfieldx - 1) * gridSize()), enemyYPosition() + ((m_enemyfieldy - 1) * gridSize()));
+        return QRect(enemyXPosition(), enemyYPosition(), m_enemyfieldx * gridSize(), m_enemyfieldy * gridSize());
 }