Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align testpilotcloud themeing with original ownCloud (except for name and artwork) #6685

Closed
jnweiger opened this issue Aug 2, 2018 · 6 comments
Assignees
Labels
ReadyToTest QA, please validate the fix/enhancement
Milestone

Comments

@jnweiger
Copy link
Contributor

jnweiger commented Aug 2, 2018

openSUSE Leap 42.2 and Fedora 28, testpilotcloud-client-2.5.0~beta1

When entering demo.owncloud.org, demo, demo in the firstrun wizard, The connect page opens.
and the user can choose several sync options.

grafik
Ontop of the connection page, the 'choose what to sync' popup immediately appears, with everything deselected. This popup should only appear, If I click 'Choose what to sync' and it should start with everything selected.

When dragging the popup a bit downwards, one can see that the radio button is at 'Synchronize everything' -- as seen in the screenshot above.

@ogoffart
Copy link
Contributor

ogoffart commented Aug 3, 2018

Note: testpilot has different theme options. One of them is that Theme::wizardSelectiveSyncDefaultNothing is set to true:

   /**
     * When true, the setup wizard will show the selective sync dialog by default and default
     * to nothing selected
     */
    virtual bool wizardSelectiveSyncDefaultNothing() const;

So it is expected to show this dialog. Maybe the problem is that the "synchronize everything from the server" is selected. This is because we choose the right checkbox when we close that dialog. So maybe i can fix it so the right radio button is selected.

@ogoffart ogoffart self-assigned this Aug 3, 2018
ogoffart added a commit that referenced this issue Aug 3, 2018
... when the wizardSelectiveSyncDefaultNothing branding option is set

Note: bormally the right option is set when the "choose what to sync"
dialog is closed.  So this was only a problem when the dialog was visible
for the first time, the wrong option would be selected underneath.

Issue #6685
@ogoffart
Copy link
Contributor

ogoffart commented Aug 3, 2018

#6686 will make sure that the "Choose what to sync" radio button is selected in that case.

@jnweiger
Copy link
Contributor Author

jnweiger commented Aug 3, 2018

Hmm, thanks for the explanation and the fix for the button logic.
But then, the fault is in the testpilot theme: It is meant for testing, so that users can explore a new client version before they upgrade their produciton client.
Its behaviour must not intentionally differ.

@ogoffart
Copy link
Contributor

ogoffart commented Aug 3, 2018

I think this was changed so this particular option could be tested, but was never reverted back.

In general, some branding options are different in the test client so they can be tested.
But that's true that it does not make so much sense as people have been complaining about that already that the behavior is different. (This option, and some other)
So we probably should remove most of the differences.

I don't have access to the source code of the test client, so i can't fix it.

ogoffart added a commit that referenced this issue Aug 14, 2018
... when the wizardSelectiveSyncDefaultNothing branding option is set

Note: bormally the right option is set when the "choose what to sync"
dialog is closed.  So this was only a problem when the dialog was visible
for the first time, the wrong option would be selected underneath.

Issue #6685
@ogoffart ogoffart assigned jnweiger and unassigned ogoffart Aug 21, 2018
@ogoffart
Copy link
Contributor

@jnweiger , I consider that the "wrong checkbox" state is fixed.
If you want to make the theming of the testpilot client the same, you need to do it yourself for i don't have access to it.

@ogoffart ogoffart changed the title linux client pops up 'Choose what to sync' with 'Sync everything' testpilot client pops up 'Choose what to sync' with 'Sync everything' Aug 21, 2018
@guruz guruz changed the title testpilot client pops up 'Choose what to sync' with 'Sync everything' Align testpilotcloud themeing with original ownCloud (except for name and artwork) Aug 21, 2018
@guruz guruz added this to the 2.5.0 milestone Sep 5, 2018
@guruz guruz added the ReadyToTest QA, please validate the fix/enhancement label Sep 5, 2018
@jnweiger
Copy link
Contributor Author

jnweiger commented Nov 6, 2018

tested on a fresh Ubuntu 18.04

rm -rf ~/.config/ownCloud\ Testpilot\ Edition
rm -rf ~/testpilotcloud
testpilotcloudcmd -v
testpilotcloud version 2.5.1daily20181031 (build 10426)

The connection wizardnow starts normally. (*) Synchronize everything is preselected. -> OK
When clicking on ( ) [Choose what to sync] the dialog opens with all elements selected. -> OK
This is consistent with the normal owncloud client -> OK

PASS

@jnweiger jnweiger closed this as completed Nov 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ReadyToTest QA, please validate the fix/enhancement
Projects
None yet
Development

No branches or pull requests

3 participants