Skip to content

Adjust acceptance tests to changes in public share page menu - #12352

Merged
MorrisJobke merged 1 commit into
masterfrom
adjust-acceptance-tests-to-changes-in-public-share-page-menu
Nov 8, 2018
Merged

Adjust acceptance tests to changes in public share page menu#12352
MorrisJobke merged 1 commit into
masterfrom
adjust-acceptance-tests-to-changes-in-public-share-page-menu

Conversation

@danxuliu

Copy link
Copy Markdown
Member

The Download item in the menu of public share pages is no longer shown in wide (>768px) windows; the element is anyway in the DOM, so the acceptance tests now check that the item is there but hidden.

The "Download" item in the menu of public share pages is no longer shown
in wide (>768px) windows (although the element is in the DOM and shown
if resized to a narrow window).
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliudanxuliu added the 3. to review Waiting for reviews label Nov 8, 2018
@danxuliudanxuliu added this to the Nextcloud 15 milestone Nov 8, 2018
@danxuliudanxuliu mentioned this pull request Nov 8, 2018
29 tasks
@MorrisJobkeMorrisJobke mentioned this pull request Nov 8, 2018
24 tasks

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense 👍

@MorrisJobke
MorrisJobke merged commit fe261c4 into masterNov 8, 2018
@MorrisJobke
MorrisJobke deleted the adjust-acceptance-tests-to-changes-in-public-share-page-menu branch November 8, 2018 21:44
@danxuliu
danxuliu restored the adjust-acceptance-tests-to-changes-in-public-share-page-menu branch June 12, 2019 13:49
@danxuliu
danxuliu deleted the adjust-acceptance-tests-to-changes-in-public-share-page-menu branch June 12, 2019 14:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@danxuliu@rullzer@MorrisJobke@skjnldsv