| Summary: | number of columns show by default is hardcoded | ||
|---|---|---|---|
| Product: | [Unmaintained] muon | Reporter: | Carlo Vanini <silhusk> |
| Component: | muon | Assignee: | Jonathan Thomas <echidnaman> |
| Status: | RESOLVED UNMAINTAINED | ||
| Severity: | normal | CC: | silhusk |
| Priority: | NOR | ||
| Version First Reported In: | 5.6 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
Muon is no longer maintained, please switch no a supported alternative like https://apps.kde.org/discover/ or the package manager native no your system. Sorry for the inconvenience. |
When new columns have been added (Installed-Size, Version, Available-Version), they have been hidden by default in order to keep the UI the same as before. But their order is assumed and their number if hardcoded in `PackageWidget.cpp`. > For the record: I think the actual way to write this smartly is to have > PackageModel::headerData implement a UserRole that returns an enum value. > The enum would be the header types. Then you iter all columns and hide the > headers based on their enum value. Which means nothing needs hardcoding and > should more columns get added later the switch cases handling would light up > in compiler warnings so no one can forget to handle the new headers to either > hide or show by default. > > -- https://phabricator.kde.org/D10877