Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

【deepin_V23_beta3】【一般】【立即】【集成测试】【DDE】【launchpad】全屏模式中，每一页应用图标均未占满页面，存在空白位 #6286

Closed
SuperEffie opened this issue Nov 27, 2023 · 10 comments
Closed

【deepin_V23_beta3】【一般】【立即】【集成测试】【DDE】【launchpad】全屏模式中，每一页应用图标均未占满页面，存在空白位 #6286

SuperEffie opened this issue Nov 27, 2023 · 10 comments
Labels
P1 高优先 回归通过 回归通过 王子冲 王子冲

Comments

@SuperEffie
Copy link

SuperEffie commented Nov 27, 2023 •
edited by luodeepin
Loading

SUMMARY | 问题概要

【关联集成】：issue: #6270

SOFTWARE/OS VERSIONS | 软件/系统版本信息

镜像地址：https://cdimage.uniontech.com/community/releases/23-Beta2/deepin-desktop-community-23-Beta2-amd64.iso
版本信息：dde-launchpad：0.2.2

PRE-REQUIREMENT | 预置条件

No response

STEPS TO REPRODUCE | 复现步骤

  1. 添加集成仓库，下载最新版dde-launchpad
  2. 使用命令sudo apt install dcc-insider-plugin下载相关包，注销后重新登录进入桌面
  3. 控制中心-更新-技术预览选择预览版启动器后重启
  4. 点击任务栏中启动器图标，打开启动器
  5. 点击最大化按钮，启动器处于全屏模式，观察全屏模式下每一页应用图标显示

OBSERVED RESULT | 观察到的结果

全屏模式下，应用图标均未占满
launchpad图标

EXPECTED RESULT | 期望的结果

允许页面存在空白图标，同时停留 400ms允许图标拖拽到上/下一屏

ADDITIONAL INFORMATION | 额外补充

No response
@SuperEffie SuperEffie added the P3 一般优先 label Nov 27, 2023
@babyfengfjx babyfengfjx assigned BLumia and unassigned babyfengfjx Nov 27, 2023
@babyfengfjx
Copy link

@BLumia 请进行优化~

@SuperEffie SuperEffie mentioned this issue Nov 27, 2023
Closed
@kt286
Copy link

kt286 commented Nov 27, 2023 •
edited
Loading

复现方式：

  1. 命令行卸载某页上的应用，此时图标还在
  2. pkill dde-launchpad 此时图标消失 最后边出现空白 卸载几个应用就出现几个空白

@BLumia
Copy link
Member

BLumia commented Nov 28, 2023

复现方式：

  1. 命令行卸载某页上的应用，此时图标还在
  2. pkill dde-launchpad 此时图标消失 最后边出现空白 卸载几个应用就出现几个空白

2 这个是已知的（其实目前是故意的）。目的是预留允许把图标拖拽到另一屏时，当前屏幕最后可以留空的支持，所以没有做补位。拖拽功能写完后会根据时间和需求复杂程度确认是改成 v23 launcher 的行为还是新的行为。目前这个 issue 是按照旧版 v23 launcher 的行为需求提的。

@BLumia
Copy link
Member

BLumia commented Dec 15, 2023 •
edited
Loading

已和 shule 确认且与 @meiyixiang 同步确认结果：允许页面存在空白图标，同时允许图标拖拽到上/下一屏，即逻辑和很多手机 launcher 以及 macOS 的 launchpad 一致。

（注：当前集成的版本中尚不支持拖拽图标到前/后一屏，也不会自动移除空白页面。这些问题可以当 bug 提）

@BLumia BLumia assigned SuperEffie and unassigned BLumia Jan 8, 2024
@BLumia
Copy link
Member

BLumia commented Jan 8, 2024

可以在0.4.0上根据上面的结论回归。

@luodeepin
Copy link

验证失败
图标不能拖拽到上/下一屏

6286

launchpad 0.4.0
https://cdimage.uniontech.com/daily-iso/image-beige/CUSTOM/lichenggang/20240110/deepin-23-beige-beta3-amd64-20240110-150013.iso

@luodeepin luodeepin added P1 高优先 and removed P3 一般优先 labels Jan 10, 2024
@luodeepin luodeepin assigned BLumia and unassigned SuperEffie Jan 10, 2024
@BLumia
Copy link
Member

BLumia commented Jan 10, 2024

同步目前的结论：拖拽到两侧后停留延迟应为 400ms

另外目前格子视图最左侧与最右侧不应当留空，需一并修复。

预计在本周五的集成中修复。

BLumia added a commit to linuxdeepin/dde-launchpad that referenced this issue Jan 11, 2024
@BLumia
fix: make icon DnD in FullscreenFrame easier
08a6434 
原本因为最底部组件的做右侧 padding，导致图标区域并未铺满屏幕两侧，使
拖拽到最左侧或者最右侧时即视为离开了图标区域，致使无法翻页。此提交调
整了布局结构来解决此问题，使拖拽过程变得更符合直觉。

Issue: linuxdeepin/developer-center#6286
Log:
BLumia added a commit to linuxdeepin/dde-launchpad that referenced this issue Jan 11, 2024
@BLumia
fix: allow Shift+Alphabetic key to quick search
69eac34 
支持使用 Shift+字母键快速开始搜索（最终会输入为大写字母）

Issue: linuxdeepin/developer-center#6286
Log:
BLumia added a commit to BLumia/dde-launchpad that referenced this issue Jan 12, 2024
@BLumia
chore: release dde-launchpad 0.4.2
eaab8be 
Changelog:

  * Adjust section title font size in WindowedFrame (linuxdeepin/developer-center#6208)
  * Fix context menu did not get hide (linuxdeepin/developer-center#6716)
  * Make icon DnD in FullscreenFrame easier (linuxdeepin/developer-center#6286)
  * Allow Shift+Alphabetic key to quick search (linuxdeepin/developer-center#6859)
  * Fix missing icon for uninstall (linuxdeepin/developer-center#6686)
  * Update page indicator when page count changed (linuxdeepin/developer-center#6861)

Log:
@BLumia BLumia mentioned this issue Jan 12, 2024
Merged
@deepin-bot deepin-bot bot mentioned this issue Jan 12, 2024
Closed
@luodeepin
Copy link

验证通过
6286

@SuperEffie 请在最新镜像中回归验证

@luodeepin luodeepin assigned SuperEffie and unassigned BLumia Jan 15, 2024
@SuperEffie
Copy link
Author

SuperEffie commented Jan 16, 2024 •
edited
Loading

【镜像地址】：https://cdimage.uniontech.com/daily-iso/image-beige/CUSTOM/lichenggang/20240116/deepin-23-beige-beta3-amd64-20240116-105126.iso
【版本信息】：dde-launchpad：0.4.2
【验证结论】：测试通过，关闭issue

  1. 应用图标可正常拖动
  2. 启动器页面中空白项是允许存在的

【录屏信息】：
tuodong

@SuperEffie SuperEffie removed their assignment Jan 16, 2024
@SuperEffie SuperEffie added P2 中优先 赵小倩 回归通过 回归通过 王子冲 王子冲 and removed P2 中优先 labels Jan 16, 2024
@SuperEffie SuperEffie moved this to CLOSED in V23 Beta2 Jan 16, 2024
@SuperEffie
Copy link
Author

【备注信息】：扩充场景：任务栏位置设置为“左”/“右”时，出现应用图标无法拖动到前置/后置页面的情况

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P1 高优先 回归通过 回归通过 王子冲 王子冲
Projects
V23 Beta2
Status: CLOSED
Milestone
No milestone
Development

No branches or pull requests
5 participants
@BLumia @kt286 @babyfengfjx @SuperEffie @luodeepin