• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

openSUSE / agama / 8925013654
72%

Build:
DEFAULT BRANCH: master
Ran 02 May 2024 01:53PM UTC
Jobs 3
Files 756
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

02 May 2024 01:50PM UTC coverage: 75.046% (-0.03%) from 75.077%
8925013654

push

github

web-flow
Storage: show skeletons only when needed (#1171)

## Problem

- The loading skeletons are displayed always, even when not needed
- The installation device selection popup is empty when clicked quickly
before loading all data

## Solution

### Loading Skeletons

- Pass the name of the component which is changed by user to parent
- Send back the changing component to all children
- There is mapping which defined whether a child needs refresh when some
setting is being changed

### Installation Device Progress

- Use the sent data when when the stored values are `undefined`
- Show a simple skeleton

### screenreaderText Properties

- I have removed all `screenreaderText` properties for skeletons and
keep it only for the result section
- The reason is that it does not make sense to notify users about 5
values in progress, it usually takes about 2 seconds to refresh the data
so the screen reader would not have enough time to read all messages
- Use it only the result section which is refreshed always, moreover the
data for the other values will be read together with the result data,
they will always take the same time

## Questions

- Do we need more granularity with changing values? I.e. if the Btrfs
snapshots are enabled/disabled then the partition list below is refresh
as well. (IMHO fine to solve this later in a separate card / PR.)
- Better progress for the installation device loading? (Note: do not
overengineer here, the progress is displayed only when you click the
link very quickly after opening the storage page, in most cases it won't
be used at all.)

## Testing

- Tested manually

## Screenshots

When enabling/disabling the Btrfs snapshots all data was refreshed, now
the installation device and the space policy does not show the progress
skeletons.

| Original | Fixed |
|---|---|
|
![storage_skeletons_broken](https://github.com/openSUSE/agama/assets/907998/9833814b-ee02-42c3-be7f-089f4b832cba... (continued)

1986 of 2867 branches covered (69.27%)

Branch coverage included in aggregate %.

20 of 35 new or added lines in 8 files covered. (57.14%)

26 existing lines in 5 files now uncovered.

18527 of 24467 relevant lines covered (75.72%)

34.77 hits per line

Subprojects
ID Flag name Job ID Ran Files Coverage
3 web 8925013654.3 02 May 2024 01:53PM UTC 0
75.29
GitHub Action Run
3 service 8884191359.3 29 Apr 2024 07:20PM UTC 0
85.88
GitHub Action Run
3 rust 8663344105.3 12 Apr 2024 02:13PM UTC 0
41.59
GitHub Action Run
Source Files on build 8925013654
Detailed source file information is not available for this build.
  • Back to Repo
  • Github Actions Build #8925013654
  • a554ab0c on github
  • Prev Build on master (#8924356118)
  • Next Build on master (#8937163624)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc