Revert "feat: hide compact mode"

[mhduiy]

This reverts commit fd30c4f.

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kegechen, mhduiy

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

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

[mhduiy]

/test all

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码重构：

   • 将compactDisplayModule的创建和初始化逻辑提取到一个单独的函数中，可以提高代码的可读性和可维护性。例如，可以创建一个名为createCompactDisplayModule的函数，将创建和初始化WidgetModule<SwitchWidget>的逻辑放在其中。

2. 内存管理：

   • 在新代码中，compactDisplayModule被直接创建并传递给appendChild函数，而没有使用new关键字进行显式内存分配。这可能会导致内存泄漏，因为appendChild函数可能没有正确管理内存。建议使用智能指针（如std::unique_ptr）来管理动态分配的对象，以确保它们在不再需要时被正确释放。

3. 代码重复：

   • compactDisplayModule和compactDisplayTipModule的创建和初始化代码非常相似，可以考虑将相似的部分提取到一个通用的函数中，以减少代码重复。

4. 可见性设置：

   • 在旧代码中，compactDisplayModule和compactDisplayTipModule的可见性被显式设置为false。在新代码中，compactDisplayTipModule的可见性设置被移除了。如果这是有意为之，应该添加相应的注释说明原因。如果这是疏忽，应该恢复可见性设置。

5. 连接信号和槽的分离：

   • 在新代码中，信号和槽的连接逻辑被直接嵌入到[this]捕获块中。虽然这样做可以减少代码行数，但可能会降低代码的可读性和可维护性。建议将信号和槽的连接逻辑提取到单独的函数中，以提高代码的可读性。

6. 翻译字符串的重复使用：

   • switchButton->setTitle(tr("Compact Display"));和label->setText(tr("If enabled, more content is displayed in the window."));中的翻译字符串tr("Compact Display")被重复使用。建议将翻译字符串提取到一个变量中，以减少重复。

7. 注释和文档：

   • 代码中没有足够的注释来解释compactDisplayModule和compactDisplayTipModule的用途和功能。建议添加适当的注释，以便其他开发者能够更容易地理解代码的目的和逻辑。

综上所述，建议对代码进行重构，以提高代码的可读性、可维护性和性能。同时，注意内存管理、代码重复和注释等方面的问题，确保代码的质量。

[mhduiy]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)