XMuli / ChineseChess

Cross-platform and online battle platform game based on Qt: Chinese Chess. Also known as:『Xiangqi』『中国象棋』
https://www.bilibili.com/video/av45509758
GNU General Public License v3.0
709 stars 186 forks source link

使游戏界面可以随窗口大小改变 #2

Closed Bruce-Ch closed 4 years ago

Bruce-Ch commented 4 years ago

新增两个commit,调整了ui设计(总体观感应当同原有相似),修改paintEvent,使得游戏界面能够随窗口大小改变;同时修改了鼠标点按事件,使游戏进程可以正常进行

XMuli commented 4 years ago

且, 若是希望尽快回复,是可以@ 我的(或指定代码审查者),否则没有关注此仓库,是看不到有人提交 pr 的

Bruce-Ch commented 4 years ago

谢谢您的建议,我受益匪浅。

XMuli commented 4 years ago

看到提交是使用设计师的布局来修改的棋盘布局的最大化最小化问题;其实还是有改进的地方的,限制最小的值,缩小到一定时候不可在最小化,可以避免 QLable 很小, 而右侧显示很大的问题;但是都很大的时候,是没有问题的;

另外,再提供一个思路, 设置棋盘同比例变大变小的话, 是可以使用棋盘的 widgetheight 来减去 右侧计时、减去棋盘四周的 QMargins, 然后除以总的 格子数目(棋子的大小同理也要修改)

ChineseChess.pro.user 文件删除掉后, 提交一下, 一起合并进来~

XMuli commented 4 years ago

感谢你的贡献,待会将你添加到贡献者里面

XMuli commented 4 years ago

能够再做一次补充提交? 将 commit 信息按照规范提交上来(feat/fix/docs: 英文表述), 再合并一次,(不然主页显得有点乱)?

Bruce-Ch commented 4 years ago

我很愿意这样做,但我现在遇到了一些困难。 为了首先将我fork得到的仓库和您的仓库同步,我首先在我自己的仓库中进行了一次merge,之后再尝试提起pull request的时候我发现我将提交5个commit,包括了我之前的提交、和您的仓库的同步以及新的提交。 不知道我是否应当采取什么别的措施,或是这样提交就可以了

Bruce-Ch commented 4 years ago

此外,我想确认一下我的此次修改应当被视为feat还是fix @xmuli

XMuli commented 4 years ago

这是 git 使用问题; 你可以如下操作:

git stash  //存储当前文件
git remote add other git@github.com:xmuli/chinessChess.git  //关联其他云仓库
git pull other master --rebase //更新仓库
git stash pop //恢复你之前的工作状态 
继续修改后提交

关于 git 的使用可以参考 学习-git 或者其他自行搜索

XMuli commented 4 years ago

此外,我想确认一下我的此次修改应当被视为feat还是fix @xmuli

feat 表示功能性的修改, 本次提交可以为,比如 feat: Remove extra comments