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

Changed the way of listing of apps in Info-apps, Quiz apps, and Spellings app #6

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

srujanjha
Copy link
Contributor

The apps in case of Info-apps, Quiz apps, and Spellings app were displayed as filenames, which included the extension part too. For example, science.txt, prepositions.txt. This was fixed.
screenshot_2015-03-08-14-18-10

There was a small issue. On pressing the "Next" button, while the last Flash Card is on display, was making the app to stop unfortunately.
This issue has been fixed.
"Unfortunately Stopped" issue fixed
…zzle, and also the scorecard.

ScoreActivity was meant only for Quiz and not for Spellings puzzle. It
was not able to handle the scores of Spellings puzzle. The issue was
fixed. Now ScoreActivity can handle scores from both Quiz and Spellings
Puzzle.
Also, there was an issue with restarting the Spellings Puzzle, the
scores start accumulating and also the index of the question was not set
to '0' while restarting the puzzle.
…lings Puzzle, and also the scorecard."

This reverts commit 1fceec1.
…uzzle.

ScoreActivity was meant only for Quiz and not for Spellings puzzle. It
was not able to handle the scores of Spellings puzzle. The issue was
fixed. Now ScoreActivity can handle scores from both Quiz and Spellings
Puzzle.
Also, there was an issue with restarting the Spellings Puzzle, the
scores start accumulating and also the index of the question was not set
to '0' while restarting the puzzle.
The apps in case of Info-apps, Quiz apps, and Spellings app were
displayed as filenames, which included the extension part too. For
example, science.txt, prepositions.txt. This was fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant