fix: Remote server item text overlaps with buttons

linuxdeepin / deepin-terminal

deepin-terminal-reborn

GNU General Public License v3.0

143 stars 68 forks source link

fix: Remote server item text overlaps with buttons #342

Closed ArchieMeng closed 5 months ago

ArchieMeng commented 5 months ago

Extend elidedText limits when being hovered at

Issue: https://github.com/linuxdeepin/developer-center/issues/7690 Log: Remote server item text overlaps with buttons

deepin-ci-robot commented 5 months ago

deepin pr auto review

关键摘要:

在ItemWidget::paintEvent函数中，对QString的赋值应该在if判断的内部，以确保在m_funcButton可见时不会进行不必要的计算。
Utils::getElidedText函数的使用可能会导致性能问题，因为每次paintEvent调用都会计算文本的缩进，如果这个函数可以避免不必要的计算，应该考虑优化。

合并相似的问题是否建议立即修改:

deepin-ci-robot commented 5 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/deepin-terminal/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment