forked from se-edu/addressbook-level4
-
Notifications
You must be signed in to change notification settings - Fork 69
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
[v1.4][W09-B4] Flashy #98
Open
jethrokuan
wants to merge
130
commits into
nus-cs2103-AY1718S2:master
Choose a base branch
from
CS2103JAN2018-W09-B4:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[v1.4][W09-B4] Flashy #98
jethrokuan
wants to merge
130
commits into
nus-cs2103-AY1718S2:master
from
CS2103JAN2018-W09-B4:master
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- update NFRs - update Target User Profile
* Update travis URL * Update link references in Developer Guide
* Add Card model * Use front and back instead of question and answer
* Add theme changing feature * Change CSS loading implementation in MainWindow class
* Add UniqueCardList * Add CardBook * fix style errors * add newlines * fix style errors
This is because the current theme affects components beyond the MainWindow.
* Add an add card feature * Minor changes * Fix style issues * Fix tests issues * Fix style errors
* Add help prompts for adding card and changing theme * Add help prompt for quitting the application * Update command summary * Fix typo error * Update introduction paragraph
* Add UUID Predicate for Card * Add UUID to card class * Add list card command
* Update README
* Add GUI tests for CardCard * add GUI tests for CardListPanel * fix style errors
* Add tests for theme changing functionality * Fix style issues * Refactor MESSAGE_INVALID_THEME logic message * Refactor parseTheme method in ParserUtil
* Remove tag functionality from addressBook * fix test
* Refactor Person to Tag - BrowserPanel related items have been left untouched, because they will probably be deleted - User and Developer Guide not updated * fix style error
* Remove email field from Tag * Remove phone field from Tag * Address -> Description * update test items to proper tags * increase timeout for GUI bot to prevent test run failure * add more address -> description cleanup * fix style errors * one more! * add more address -> description
* update guava version * add CardTag class * Delete that distracting blank line in Card.java
* Tweak UI to show option value for back for card and show numbered list for options, and add EditCardCommandSystemTest * Improve CardCard tests
* Add selectc to user guide * Add portfolio * Fix style errors
…c cards and storage (#200)
* Fix colours * Fix card back * Update ui image * Change to default light theme * Fix typo * Update portfolio and UGDG * Add collation markers
* Change outdated comment * Enhance dark theme
* Update developer guide for themes * Add swimlane diagram * Add autocomplete docs
* UserGuide update. add preface, reorder some, add logo * Clean up on Logo
* Update User Guide * fix based on @fooyongjie's comments
* refractor date tests * refactor date test
* Rename seedu.address to seedu.flashy * Rename AddressBook
* update userguide summary showdue, answer, schedule * change summary to sentence case
* Tweak 'addc' section of UG, add authorship * Fix issues
* Populate FAQ * Modify ppp * Modify enhancements in dev guide * Fix formatting * Rename to flashy * Modify user guide * Remove capitalisation of flashy
* Remove encryption * Collate code
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.