Skip to content

window sizeを2倍に変更#19

Open
hirarira wants to merge 11 commits into
masterfrom
feature/spread_edit_map
Open

window sizeを2倍に変更#19
hirarira wants to merge 11 commits into
masterfrom
feature/spread_edit_map

Conversation

@hirarira
Copy link
Copy Markdown

@hirarira hirarira commented May 1, 2019

window sizeが440*460とマジックナンバーになっていたのを、定数に置き換えました。
SCREEN_CHIP_SIZEを21にして、横マスに21オブジェクトを表示し、840px * 860px 出力にしました。

@hirarira hirarira requested a review from aokashi May 1, 2019 11:09
@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 1, 2019

解像度が低い端末向けに、デフォルトでは従来どおりとし、
ボタンによって拡大表示が出来るように修正しました。

微妙にサイズ噛み合ってないので要修正

@aokashi
Copy link
Copy Markdown
Member

aokashi commented May 1, 2019

📝 懸念点

  • マップ画面のコピーやペーストは正常にできるか
  • 使い勝手を考えると、以下の方法で実装したいが、実現可能か
    • マップ編集ウインドウをサイズ変更可能にし、サイズ変更するとその分マップ編集サイズも変わるようになる
    • マップ編集ウインドウや物体パーツ選択ウインドウ、背景パーツ選択ウインドウのそれぞれのサイズや位置を記憶し、起動後にその記憶した位置やサイズを復元できるようになる (サイズや位置は WinWwamk.ini に記録すること)

@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 1, 2019

  • マップ画面のコピーやペーストは正常にできるか

出来ましたね

@aokashi
Copy link
Copy Markdown
Member

aokashi commented May 1, 2019

  • マップ画面のコピーやペーストは正常にできるか

出来ましたね

む、自分だと1画面のウインドウサイズで行ったところ、2画面分でペーストされてしまいました。

@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 2, 2019

1画面の時は1画面だけのほうが良いっすよね・・・
という訳で現在の表示領域に合わせてコピー範囲を切り替えました

@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 2, 2019

物体パーツウィンドウなど、各種ウィンドウを拡大縮小したときに付いてくるように修正します

@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 2, 2019

↑対応しました。
良ければレビューお願いします 🙏

Copy link
Copy Markdown
Member

@aokashi aokashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメント加えました。

Comment thread WinWwamk.cpp Outdated
int setScreenChipSize = g_hugeMapSize ? DEFALUT_SCREEN_CHIP_SIZE : SCREEN_CHIP_SIZE;
setScreenChipSize++;
g_hugeMapSize = !g_hugeMapSize;
SizeX = (setScreenChipSize * CHIP_SIZE);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

サイズ変更したところ、横幅と縦幅に若干余白が発生していました。
この場合は GetWindowRectGetClientRect の差分を差し引いてサイズを計算するとぴったり調整できるかもしれません。
(WinMain関数の処理を見ると多分分かるかもしれないです)

Comment thread WinWwamk.cpp Outdated
// �ݒ�t�@�C���ǂݍ���
PositionX = GetPrivateProfileInt("Main", "PositionX", CW_USEDEFAULT, g_szSettingFile);
PositionY = GetPrivateProfileInt("Main", "PositionY", 0, g_szSettingFile);
int window_size = SCREEN_CHIP_SIZE * CHIP_SIZE;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

window_size は利用されていますか・・・?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あっ・・・使ってないので消しちゃってください 🙏

Comment thread WinWwamk.cpp
MoveWindow(g_hDlgQuickView, rectBox.left, rectBox.bottom, rect.right - rect.left, rect.bottom - rect.top, TRUE);
ShowWindow(g_hDlgQuickView, SW_SHOW);
// ���́E�w�i�I���p�[�c�E�B���h�E�ʒu�̃��Z�b�g
resetSelectMapAndObjectPosition();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@aokashi aokashi changed the title window sizeを2倍に変更 [3.2.x] Feature: window sizeを2倍に変更 May 6, 2019
@aokashi aokashi changed the title [3.2.x] Feature: window sizeを2倍に変更 window sizeを2倍に変更 May 6, 2019
@aokashi aokashi added this to the v3.2.0 milestone May 6, 2019
@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 6, 2019

2点修正しましたー

@aokashi
Copy link
Copy Markdown
Member

aokashi commented May 6, 2019

あっ・・・
image

@hirarira hirarira force-pushed the feature/spread_edit_map branch from ff231a1 to 29b20a7 Compare May 6, 2019 15:30
@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 6, 2019

  • キーボードキーによるスクロール
  • マウスによるスクロール
  • ボタンによるスクロール
    の3点に対応する必要あり

拡小時に端までスクロールできない問題
@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 6, 2019

細かい点だとスクロール位置も合っていない

@hirarira
Copy link
Copy Markdown
Author

hirarira commented May 7, 2019

スクロールは対応したので、残りはスクロール位置の対応のみ

@aokashi
Copy link
Copy Markdown
Member

aokashi commented Sep 19, 2020

ミニマップ機能の実装 (#18) については、パーツの矩形配置後ラグが生じており、常用するにはまだ難しい状態なので、ミニマップよりも先にこのブランチを適用するかもしれないです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants