Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

[OYPD-468] Add abilty to stop the contious scroll of the slides in the gallery. #545

Merged
merged 1 commit into from Jun 14, 2017
Merged

[OYPD-468] Add abilty to stop the contious scroll of the slides in the gallery. #545

merged 1 commit into from Jun 14, 2017

Conversation

ivan-berezhnov
Copy link
Contributor

@ivan-berezhnov ivan-berezhnov commented Jun 12, 2017

Make sure these boxes are checked before asking for review of your pull request - thank you!

General checks

  • All coding styles are fulfilled and there are no any issues reported by CodeSniffer CI.
    CI code sniffer errors
  • All tests are running and there are no failed tests reported by CI.
    Behat test results
  • Documentation has been updated according to PR changes.
  • Steps for review have been provided according to PR changes.
    Steps for review
  • Make sure you've provided all necessary hook_update_N to support upgrade path.
  • Make sure your git email is associated with account on drupal.org, otherwise you won't get commits there.
    drupal.org email
  • If you would like to get credits on drupal.org, check documentation.

Thank you for your contribution!

Jira issue:

https://propeople-us.atlassian.net/browse/OYPD-468
Drupal.org issue: https://www.drupal.org/node/2885456

Steps to review:

  • Go to [sitename]/accelerator
  • Click "tab" on the keyboard
  • When you have focused on the Left Arrow or Right
  • Verify slider is stopped
  • Click "tab" on the keyboard again and unfocus the Left Arrow or Right
  • move mouse cursor on the gallery or the Left Arrow or Right buttons
  • Verify slider is stopped

@ivan-berezhnov ivan-berezhnov self-assigned this Jun 12, 2017
@ivan-berezhnov ivan-berezhnov added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Jun 12, 2017
@ivan-berezhnov ivan-berezhnov added this to the Version 1.3 milestone Jun 12, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 13, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 13, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 14, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 14, 2017
@ymcatwincities ymcatwincities deleted a comment from ivan-berezhnov Jun 14, 2017
@Sanchiz Sanchiz added PR: Code Reviewed Code reviewed. Ready for QA and removed PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Jun 14, 2017
@gundevel
Copy link
Collaborator

Build comment file:

===============

Environment Link
Fresh OpenY installation http://ci.openymca.org/build1523
Upgraded(upgrade path) installation http://upgrade.openy-dev.ffwua.com/build1523
Installation process http://install.openy-dev.ffwua.com/build1523/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://ci.openymca.org:8080/job/PR_BUILDER/1523/

@gundevel
Copy link
Collaborator

@Sanchiz Sanchiz merged commit 262d372 into ymcatwincities:8.x-1.x Jun 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Code Reviewed Code reviewed. Ready for QA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants