banji-project / ring-issues

Old issues before it was moved to kde.org
0 stars 0 forks source link

Settings: Add option "Account Wizard" which brings up the wizardscreen again from the ring-kde client #4

Closed star-buck closed 7 years ago

star-buck commented 7 years ago

this option could be added in the menu bar -> Settings -> another entry "Account Wizard" at the bottom of the dropdown menu options

Elv13 commented 7 years ago

pick11

Done

(as always for beta of any kde apps, rm ~/.local/share/kxmlgui5/ring-kde/ring-kdeui.rc may be necessary)

star-buck commented 7 years ago

Comes up nicely here. To simplify and make consistent, in this case I would right away start with the Wizard page, and skip the welcome page layer, so this window comes up straight: screenshot_20170421_170035

star-buck commented 7 years ago

@Elv13 : What about the last comment proposal?

Elv13 commented 7 years ago

The wizard welcome page has been disabled as proposed

star-buck commented 7 years ago

So the welcome page has been disabled only in this case when directly invoked from menu, but it still shown normally while starting the client (with or without having an account already defined)?

Elv13 commented 7 years ago

Correct. The splash screen replaced the wizard welcome page (as suggested). It is shown (unless disabled in the config) when the client launch (both the first and subsequent usage). The splash screen is not shown when invoked by the menu.

star-buck commented 7 years ago

The splash screen replaced the wizard welcome page (as suggested).

This is not related to this ticket, but since you mention it here again: There should not be another splash screen though, as the Welcome splash already IS a splash screen.

Elv13 commented 7 years ago

Sorry if I misunderstood your original intent. While I did "unify" the splash screen, I did it the other way around. This can be fixed quickly.

Please note that technically, from a pure protocol point of view, the wizard isn't really considered a splash screen. I don't want to by pedantic here, but a splashscreen is "hardcoded"[1] into X11 (probably not in wayland) as a special window type designed for centered, frameless and volatile popup. The QSplashScreen isn't designed to do what you describe either. I can revert the commit and use the wizard popup on every startup and only show the first page. However, before doing so, I would like you to consider the following and take the decision:

[1] The WM (KWin) does the actual work, but the protocol specifies how

star-buck commented 7 years ago

lets go with the welcome splash as described #3 for several reasons (but also if only for the reason the other is X11-dependent anyway)

star-buck commented 7 years ago

since this ticket about the option to show the wizard again is solved, ill close this here now.