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.

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

【deepin_V23_Release】【一般】【立即】【预发布测试】【DDE】【UI】企业微信图标偏小 #9572

Closed
wangruoxuan3782 opened this issue Jul 5, 2024 · 5 comments
Labels
回归通过 回归通过 蒙宇韬 蒙宇韬 许琪 许琪

Comments

@wangruoxuan3782
Copy link

wangruoxuan3782 commented Jul 5, 2024

SUMMARY | 问题概要

No response

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

dde-launchpad 0.7.6
dde-shell 0.0.27

PRE-REQUIREMENT | 预置条件

3840*2160 1.5倍缩放

STEPS TO REPRODUCE | 复现步骤

打开全屏启动器

OBSERVED RESULT | 观察到的结果

企业微信图标偏小
截图_选择区域_20240704164404

EXPECTED RESULT | 期望的结果

图标大小正常显示

ADDITIONAL INFORMATION | 额外补充

No response

@xuqi27837288 xuqi27837288 assigned kegechen and ArchieMeng and unassigned kegechen Jul 8, 2024
@xuqi27837288
Copy link

@ArchieMeng

@xuqi27837288 xuqi27837288 assigned ArchieMeng and unassigned ArchieMeng Jul 8, 2024
@ArchieMeng
Copy link

ArchieMeng commented Jul 9, 2024

image
@xuqi27837288 分析发现是企业微信的图标没有给全对应的图标素材。参照微信的图标素材补全就好了。
需要找设计补图。

@kegechen
Copy link

kegechen commented Jul 9, 2024

bloom index.theme 下面的 apps/xxx 支持scale

[apps/96]
Size=96
Context=Applications
Type=Scalable
MinSize=16
MaxSize=512

不仅仅是96,其他大小应该都支持缩放。这个 MinSize MaxSize 可能需要参考一下其他第三方主题的设置

QIcon icon = QIcon::fromTheme("com.qq.weixin.work.deepin");
    qInfo() << "availableSizes" << icon.availableSizes();
    QList<int> sizes = {16, 24, 32, 64, 96, 128, 256, 512};
    QWidget widget;
    QHBoxLayout *lay = new QHBoxLayout(&widget);
    for (int i : sizes) {
        DIconButton * button = new DIconButton();
        button->setIcon(icon);
        button->setIconSize(QSize(i, i));
        lay->addWidget(button);
    }
    widget.show();

image

@wangruoxuan3782
Copy link
Author

UI已验证
deepin-icon-theme 2024.07.10

@xuqi27837288
Copy link

【验证环境】:v23 RC2镜像
镜像:https://cdimage.uniontech.com/community/releases/23-RC2/amd64/deepin-desktop-community-23-RC2-amd64.iso
仓库:deb https://ci.deepin.com/repo/deepin/deepin-community/testing/ unstable main dde community commercial
内核:Linux test-PC 6.6.35-amd64-desktop-hwe
【验证步骤】:问题单中描述的步骤
【验证结果】:回归测试通过

@xuqi27837288 xuqi27837288 reopened this Jul 11, 2024
deepin-bot bot pushed a commit to linuxdeepin/deepin-icon-theme that referenced this issue Jul 11, 2024
Issue: linuxdeepin/developer-center#9572
Log: some app icons are small on high-res screen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
回归通过 回归通过 蒙宇韬 蒙宇韬 许琪 许琪
Projects
None yet
Development

No branches or pull requests

5 participants