Bug 369530

Summary:	bad use of screen space on small screen
Product:	[Applications] knavalbattle	Reporter:	Mathieu Jobin <opensource>
Component:	general	Assignee:	Roney Gomes <roney477>
Status:	REOPENED ---
Severity:	minor	CC:	aacid, kde-games-bugs-null
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/games/knavalbattle/commit/7c8511c24d63357f82b4a0d0fcabf4e2ccf00e0f	Version Fixed In:
Sentry Crash Report:
Attachments:	screenshot updated screenshot modified example

Description Mathieu Jobin 2016-09-29 12:38:18 UTC

I thought I could play this game with my son.
he has a small computer with 10" screen.

I am attaching a screenshot

would be great to see better use of screen space

thank you


Reproducible: Always

Steps to Reproduce:
1. get small computer with small resolution 1024 x 600
2. launch knavalbattle



Expected Results:  
bigger board games, less gray space

Comment 1 Mathieu Jobin 2016-09-29 12:39:57 UTC

Created attachment 101345 [details]
screenshot

Comment 2 Mathieu Jobin 2021-01-02 08:05:32 UTC

Created attachment 134457 [details]
updated screenshot

on small res, there is a lot unused space...

maybe if we take the scoring boxes to the side?

Comment 3 Mathieu Jobin 2021-01-02 08:08:41 UTC

Created attachment 134458 [details]
modified example

from previous screenshot, used kolourpaint to explain my suggestion

Comment 4 Bug Janitor Service 2021-05-26 04:16:41 UTC

A possibly relevant merge request was started @ https://invent.kde.org/games/knavalbattle/-/merge_requests/3

Comment 5 criticaltemp 2021-12-09 16:23:25 UTC

Git commit 7c8511c24d63357f82b4a0d0fcabf4e2ccf00e0f by Michael Lang.
Committed on 26/05/2021 at 03:56.
Pushed by aacid into branch 'mr/3'.

Add responsive design to support smaller screens and mobile devices

Slighly smaller fonts and margins.
Adjust placement of labels and fields relative to window dimensions.
Portrait layout and landscape specific layouts.

M  +4    -3    src/playerlabel.cpp
M  +2    -2    src/playfield.cpp
M  +91   -26   src/seaview.cpp
M  +3    -2    src/seaview.h
M  +3    -4    src/statswidget.cpp
M  +0    -1    src/statswidget.h

https://invent.kde.org/games/knavalbattle/commit/7c8511c24d63357f82b4a0d0fcabf4e2ccf00e0f

Comment 6 Albert Astals Cid 2021-12-09 18:11:58 UTC

This is not fixed, the below automated message was due to a mistake in one rogue script