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

FEV-130 - IVQ, Skip Back/Next #3651

Merged
merged 4 commits into from
Feb 25, 2018
Merged

FEV-130 - IVQ, Skip Back/Next #3651

merged 4 commits into from
Feb 25, 2018

Conversation

korniiov
Copy link
Contributor

[IVQ-Player] local issue - continue used for 2 places
Without UI

[IVQ-Player] local issue - continue used for 2 places
Without UI
@korniiov korniiov requested a review from eitanavgil December 13, 2017 09:22
@KalturaGitBot
Copy link
Contributor

Live Pull Request Urls

1 similar comment
@KalturaGitBot
Copy link
Contributor

Live Pull Request Urls

Copy link
Contributor

@eitanavgil eitanavgil left a comment

Choose a reason for hiding this comment

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

@MaksymKorniiov
Once we get UI/UX we will reopen this. For now I mark this as "request changes".

@eitanavgil eitanavgil changed the base branch from 2.64 to 2.65 December 27, 2017 07:34
@eitanavgil eitanavgil changed the title FEV-130 FEV-130 - IVQ, Skip Back/Next Dec 27, 2017
@eitanavgil eitanavgil changed the base branch from 2.65 to 2.66 January 30, 2018 14:50
@eitanavgil eitanavgil changed the base branch from 2.66 to ivq-enhancements February 19, 2018 07:45
Copy link
Contributor

@eitanavgil eitanavgil left a comment

Choose a reason for hiding this comment

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

Implement new graphics

1. add imgs
2. fix position
@KalturaGitBot
Copy link
Contributor

Updated pull request 3651: 34b439b

1 similar comment
@KalturaGitBot
Copy link
Contributor

Updated pull request 3651: 34b439b


if(currentPoint.key > allPoints[0].key){
var prevQuePoint = allPoints[currentPoint.key-1];
if( (prevQuePoint && _this.KIVQModule.canSkip) || (!_this.KIVQModule.canSkip && prevQuePoint && prevQuePoint.isAnswerd)){
Copy link
Contributor Author

Choose a reason for hiding this comment

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

add comments and add fix logic (check is currentCP is answered)

1: add comments
2: change logic
@KalturaGitBot
Copy link
Contributor

Updated pull request 3651: 7e7fc59

@KalturaGitBot
Copy link
Contributor

Updated pull request 3651: f0c0886

@eitanavgil eitanavgil merged commit cdbf659 into ivq-enhancements Feb 25, 2018
@OrenMe OrenMe deleted the FEV-140 branch March 15, 2018 07:19
eitanavgil added a commit that referenced this pull request Mar 28, 2018
* 1. add feature to getting CP type
2. add new property to CP (partnerData)

* 1. change partnerData to questionType
2. rename questions type

* 1. Add const QuestionsType

* FEV-138 (#3729)

[IVQ-Player] show answers should still show questions.

* FEV-132 (#3730)

[IVQ-Player] New Q type #2: True/False

* 1. Add const QuestionsType (#3728)

* ivq-enhancements:
add opacity to reflection point on last screen

* ivq-enhancements:
fix css. add new imgs

* FEV-166: (#3736)

* FEV-166:
Support links in questions fields

* renaming function and change comments

* KMS-17072-information-screen-after-submit: (#3744)

IVQ: No information and elements when question icon clicked on Submit screen

* KMS-16785-remove-welcome-page: (#3742)

* KMS-16785-remove-welcome-page:
[IVQ-Editor] new option to remove welcome page

* KMS-16785-remove-welcome-page:
space fix

* FEV-130 - IVQ, Skip Back/Next (#3651)

* FEV-140
[IVQ-Player] local issue - continue used for 2 places
Without UI

* FEV-130
1. add imgs
2. fix position

* FEV-130 - IVQ, Skip Back/Next
1: add comments
2: change logic

* supporting banSeekManager plugin from Editor (#3703)

* supporting banSeekManager plugin from Editor

* banSeek_editorSupport
1. fix from PR comments
2. rename alertText to noSeekAlertText

* FEV-131-auto-submit-rp: (#3750)

1. when reached reflection point - auto submit it

* bump version

* FEV-172-align-mid-RP-text: (#3756)

* FEV-172-align-mid-RP-text:
1. vertical-align for RP text

* FEV-172-align-mid-RP-text:
1. fix width

* Fev 150 block portrait mode (#3761)

* FEV-150 - quiz handle portrait UX

* FEV-150_block_portrait_mode:
1. show overlay with portrait warning on mobile + vertical screens

* FEV-150_block_portrait_mode:
1. add event only if has mobile class
2. change function name

* FEV-171-design-changes (#3755)

* FEV-171-design-changes
1. change quiz-screen background
2. change font color
3. add icon for submit and review buttons

* fix backgrounds

* FEV-171-fix-designs: (#3763)

1. remove hover effect
2. wrong answers - with border

* Update DefaultSettings.php

* bump version to IVQ

* KMS-17284-submit-and-review-buttons-text-size: (#3779)

1. fix font-size and display of review and submit buttons

* KMS-17301-true-false-inline-position: (#3778)

fix margin between true/false answers

* FEV-173-skip-for-now-overlay: (#3777)

fix bottom position of footer and margin between answers

* revert version back to main branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants