louislam / uptime-kuma

A fancy self-hosted monitoring tool
https://uptime.kuma.pet
MIT License
59.65k stars 5.33k forks source link

UI: `No fil...ected.` #385

Closed jtagcat closed 3 years ago

jtagcat commented 3 years ago

image

100% zoom on HD display.

Saibamen commented 3 years ago

This is because of two columns in Settings page. I don't see a quick solution here. Even if we add width: 100% to this element, this IMO looks bad: image

jtagcat commented 3 years ago

What about having it on it's own line, and import below, align-right.

Export may get it's own section even. Tieing it with upload file to export doesn't make sense.

I think a tabbed design (default select export tab(?)), where you may select import at the top.
This may also result in hiding text, reducing bomb-overwhelm factor for the settings page.

On mobile, can't sketch atm.

Ponkhy commented 3 years ago

I'll think about something tomorrow, thank you!

Saibamen commented 3 years ago

v1:

image

v1 mobile:

image

v2:

image

v2 mobile:

image

Ponkhy commented 3 years ago

Or you just already do it xD I think we should go with v2 and also put the Import Options from #382 when it's in the master branch on the left, in the same line as the import button. Or you could submit a commit on the PR.

jtagcat commented 3 years ago

It seems I'm late to the party :)

jtagcat commented 3 years ago

meant something along the lines of https://www.w3schools.com/w3css/w3css_tabulators.asp

jtagcat commented 3 years ago

Closing for now, reopen if wanted (◕‿◕✿)

Saibamen commented 3 years ago

@jtagcat: No! This is not how we should use issues and PRs...

This issue is not fixed yet, because PRs was not merged to master branch...

jtagcat commented 3 years ago

Ah, well, that what's happens when you try to make use of free time, partial reading, thought the second PR merge was enough.

come play splatoon 1

Neer171 commented 3 years ago

How about adding CSV format to import and export option because json is not too much user friendly in order to import/export.

Saibamen commented 3 years ago

CSV requires more code for save and read. JSON is more universal. And you cannot create hierarchical and related data in CSV. No

Neer171 commented 3 years ago

how can a normal user create json file for their domains if they have 100 domains.

chakflying commented 3 years ago

@Neer171 This issue is about the import dialog design. Please open a new issue if you think another format of import should be added.