Skip to content

Improve marketplace list cards#17390

Merged
Vanessa219 merged 2 commits intosiyuan-note:devfrom
TCOTC:fix/card-style
Apr 4, 2026
Merged

Improve marketplace list cards#17390
Vanessa219 merged 2 commits intosiyuan-note:devfrom
TCOTC:fix/card-style

Conversation

@TCOTC
Copy link
Copy Markdown
Contributor

@TCOTC TCOTC commented Apr 2, 2026

Description / 描述

fix #17358 #14636 (comment)

  • 改为 grid 布局,移除末尾添加的 <div class="fn__flex-1" style="margin-left: 15px;min-width: 342px;"> 元素。筛选结果只有一个的时候也能保持正常的卡片宽度。
  • 集市包描述增加中英文字符默认间距
  • 淡化下载量图标和文本
  • 调整间距、对齐元素
  • 移除包名
  • 移除删除按钮

PixPin_2026-04-02_19-45-36

PixPin_2026-04-02_19-45-51

TODO:

Type of change / 变更类型

  • Bug fix
    缺陷修复
  • Refactoring
    代码重构
  • New feature
    新功能
  • Text updates or new language additions
    修改文案或增加新语言

Checklist / 检查清单

  • I have performed a self-review of my own code
    我对自己的代码进行了自我审查
  • I have full rights to the submitted code and agree to license it under this project's AGPL-3.0 license
    我拥有所提交代码的完整权利,并同意其以本项目的 AGPL-3.0 许可证授权
  • PR is submitted to the dev branch and has no merge conflicts
    PR 提交到 dev 分支,并且没有合并冲突


&[data-type="downloaded"] .config-bazaar__content {
margin: 16px;
}
Copy link
Copy Markdown
Contributor Author

@TCOTC TCOTC Apr 2, 2026

Choose a reason for hiding this comment

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

如果能再包一层 .b3-cards 改成多列 #17358 (comment) ,就不用这样写选择器了。

不过如果决定已下载就是单列的话,也可以加个新的 .b3-cards--nowrap 来解决。

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Bazaar (market) list card layout/styling to improve alignment and sizing, addressing issue #17358 by moving the card list to a grid-based layout and simplifying the card UI.

Changes:

  • Switched card containers from flex-wrap to CSS grid and removed the “filler” flex elements previously used to stabilize last-row sizing.
  • Simplified Bazaar list cards by removing the secondary package name display and removing the uninstall (trash) action from the market list cards.
  • Tweaked card spacing and visual emphasis (description spacing and action area styling).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
app/src/config/bazaar.ts Updates the generated Bazaar card HTML (removes wrap modifier, hides package name, removes uninstall icon) and removes filler elements used for flex layout.
app/src/assets/scss/component/_card.scss Refactors .b3-cards to a grid layout and adjusts padding/spacing; adds new icon color styling within card actions.
app/src/assets/scss/business/_config.scss Updates Bazaar config panel content layout to grid and adjusts margins for the downloaded panel content.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Vanessa219 Vanessa219 merged commit b12f0fd into siyuan-note:dev Apr 4, 2026
@TCOTC TCOTC deleted the fix/card-style branch April 4, 2026 09:38
Vanessa219 added a commit that referenced this pull request Apr 5, 2026
@Vanessa219
Copy link
Copy Markdown
Member

间距不对,已重新调整

@TCOTC
Copy link
Copy Markdown
Contributor Author

TCOTC commented Apr 5, 2026

我觉得减淡下面的图标和下载量的颜色应该挺合理的,让用户更关注集市包的名称和描述

@Vanessa219
Copy link
Copy Markdown
Member

保持一致

@TCOTC TCOTC changed the title Improve the layout style of the market list cards Improve marketplace list cards Apr 7, 2026
@TCOTC
Copy link
Copy Markdown
Contributor Author

TCOTC commented Apr 7, 2026

#17420 合并到此

@Vanessa219 Vanessa219 self-assigned this Apr 7, 2026
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.

4 participants