Skip to content

feat(web): show filesystem labels#2070

Merged
ancorgs merged 3 commits intomasterfrom
storage-label-improvements
Feb 25, 2025
Merged

feat(web): show filesystem labels#2070
ancorgs merged 3 commits intomasterfrom
storage-label-improvements

Conversation

@joseivanlopez
Copy link
Contributor

@joseivanlopez joseivanlopez commented Feb 25, 2025

The labels of the file-systems were not visible in the parts of the UI used to select a disk and/or a partition. This adds them

Before After
uno-old uno-new
dos-old dos-new
tres-old tres-new

@ancorgs ancorgs force-pushed the storage-label-improvements branch from 7d0f954 to 5369c5c Compare February 25, 2025 15:24
Comment on lines 505 to +512
<Split hasGutter>
<span>{partition.description}</span>
<span>{deviceSize(partition.size)}</span>
<SplitItem>{partition.description}</SplitItem>
{label && (
<SplitItem>
<Label isCompact variant="outline">
{label}
</Label>
</SplitItem>
Copy link
Contributor

Choose a reason for hiding this comment

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

What about still using the Flex with custom size approach?

Copy link
Contributor

Choose a reason for hiding this comment

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

Well, I like the separation given by Split here.

Copy link
Contributor

Choose a reason for hiding this comment

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

So, go ahead.

As commented in IRC, it should not be a question of taste but consistence and correct abstractions to improve maintenance and those things that we use to make so important in the backend side 😉

@coveralls
Copy link

coveralls commented Feb 25, 2025

Pull Request Test Coverage Report for Build 13525564420

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 72.658%

Totals Coverage Status
Change from base Build 13525056811: 0.0%
Covered Lines: 19890
Relevant Lines: 27375

💛 - Coveralls

@ancorgs ancorgs force-pushed the storage-label-improvements branch 3 times, most recently from c7a1217 to af75889 Compare February 25, 2025 15:47
@ancorgs ancorgs marked this pull request as ready for review February 25, 2025 16:00
@ancorgs ancorgs force-pushed the storage-label-improvements branch from df54cff to a91ec56 Compare February 25, 2025 16:03
@ancorgs ancorgs merged commit baf665d into master Feb 25, 2025
8 checks passed
@ancorgs ancorgs deleted the storage-label-improvements branch February 25, 2025 16:20
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Feb 26, 2025
https://build.opensuse.org/request/show/1248590
by user IGonzalezSosa + dimstar_suse
- Version 12

- Add fonts for CJK characters and monospaced text
  (gh#agama-project/agama#2076, bsc#1237457).

- Display the file system labels in most selectors for disks and
  partitions (gh#agama-project/agama#2070).

- Use the backend's language as fallback, ignoring the browser's one
  It reduces the chances to produce unwanted side-effects when connecting
  to the web user interface (gh#agama-project/agama#2071).
- Do not block when connecting during system installation.
- Make some small fixes/improvements to the overview, localization
  and software pages markup.

- Synchronous system probing when choosing or registering the
  product (gh#agama-project/agama#2072).

- Display network edition errors instead of navigating back
  (gh#agama-project/agama#2064).

- Ignore non-critical i
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