Missing Cockpit interfaces

Feature
,
21

Fixed:

image
image.png1659×135 6.94 KB

1 Like
22

I’d say hidden by default with progressive disclosure.

1 Like
23

Here is the new view @giacomo:

image
image.png739×682 20.1 KB

/cc: @lucag

3 Likes
24

I think the dashboard page should be titled “Webtop 5 dashboard” or just webtop 5.

For naming style reference see the Web server or Firewall apps…

1 Like
25

I noticed that some packages show Dashboard and some other the name of package. If you want I can open an improvement and change all to name of package.

26

Whatever is chosen, I think it is important to be consistent.

27

Yes in fact I can review all modules.

2 Likes
28

Opened the improvement:

@davidep I noticed also that some pages on some modules are <h1> instead of <h2>. Should I have to correct the tag in the same PRs on these pages or we can ignore that?

EDIT: I see that only modules with Dashboard wrong have also h1 tag. I fix also this.

1 Like
29

@federico.ballarini … I think maybe SMTP STARTTLS can be put in the advanced options :thinking:

In most cases it is not necessary to enable this option since communication with the server takes place in localhost and can rarely be used.

What do you think ? @amygos @giacomo

1 Like
30

@davidep I fixed the problem of dashboard name. You can see my development process into the issue #6004.

@lucag As you prefer: we can wait a feedback from @giacomo and @Amygos. But should we move under advanced settings section also SMTP Authentication?

1 Like
31

I think not @federico.ballarini , unencrypted SMTP authentication is much more used in this case :wink:

We await other opinions.

1 Like
32

IMO you can move all SMTP related option under advanced, most of the users doesn’t need to change it.
I do not even know why these options are needed :thinking:

33

SMTP authentication is used to avoid outgoing email virus scanning problems in some cases :wink:

34

Hi all, welcome back! :slight_smile:

What we want to do with SMTP options? I can finish development process and you can review the code.
Thanks

2 Likes
35

For now, just put it under advanced.
We will move it outside if many users are unable to find it :wink:

2 Likes
36

image
image.png727×632 20.4 KB

New layout. You can review code :slight_smile:

3 Likes
37

Great job @federico.ballarini :clap::clap:

I suggest inserting placeholders with the default values in the “Public URL” and “DAV server URL” fields, automatically obtaining the FQDN of the server.

For example:
WebTop%20-%20ns77ent%20neth%20eu

What do you think ?

3 Likes
38

Yes, good :smiley:

Can you check also doc-inline and labels? Then I will commit changes.

2 Likes
39

I checked and it seems to me that the labels and doc-inlines are already so good.:+1:

2 Likes
40

Excellent idea!

Wonderful team work! :clap:

3 Likes