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

feat: Abnormal background color selected in symbol plugin #105

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

tsl115
Copy link
Contributor

@tsl115 tsl115 commented Nov 24, 2023

Abnormal background color selected in symbol plugin

@tsl115 tsl115 force-pushed the origin/master branch 6 times, most recently from b912372 to 66c463c Compare November 24, 2023 04:55
@@ -35,22 +35,22 @@ SymbolTreeView::SymbolTreeView(DWidget *parent)
, d (new SymbolTreeViewPrivate(this))
{
d->model = new SymbolModel();
d->delegate = new SymbolDelegate;
DTreeView::setModel(d->model);
Copy link
Contributor

Choose a reason for hiding this comment

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

确认下:这个symboldelegate是不是没用了,是否可以删除该类

Copy link
Contributor Author

Choose a reason for hiding this comment

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

之所以这么修改是因为,如果使用了这个的话,会导致选中后的蓝色背景没有圆角,试了下应该是可以删除这个类的

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

Copy link
Contributor

@deepin-mozart deepin-mozart left a comment

Choose a reason for hiding this comment

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

请看comment

Selected effects in the rune plugin were replaced
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-mozart, tsl115

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-mozart deepin-mozart merged commit b9174da into linuxdeepin:master Nov 24, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants