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

fix: wrong height for user list frame #338

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

asterwyx
Copy link
Contributor

@asterwyx asterwyx commented Sep 15, 2023

Height for user list frame is wrong. Define some constants and make the calculation clear.

Log: fix wrong height for user list frame
Issue: linuxdeepin/developer-center#5616

@asterwyx
Copy link
Contributor Author

/integrate

@github-actions
Copy link

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#521
PrNumber: 521
PrBranch: auto-integration-6195530295

@asterwyx asterwyx force-pushed the userswitchpage branch 2 times, most recently from 07f6efc to 1f66515 Compare September 22, 2023 06:49
@asterwyx asterwyx requested a review from a team September 25, 2023 05:32
@justforlxz
Copy link
Member

/+1

src/session-widgets/userframelist.h Outdated Show resolved Hide resolved
src/session-widgets/userframelist.h Outdated Show resolved Hide resolved
Height for user list frame is wrong. Define some constants and make
the calculation clear.

Log: fix wrong height for user list frame
Issue: linuxdeepin/developer-center#5616
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: asterwyx, BLumia

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

@asterwyx asterwyx removed the approve label Sep 25, 2023
@asterwyx asterwyx merged commit 4f6aff7 into linuxdeepin:master Sep 25, 2023
18 checks passed
@asterwyx asterwyx deleted the userswitchpage branch September 25, 2023 07:46
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