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

Add bugfixes and minor features to View command UI #205

Merged
merged 10 commits into from
Nov 11, 2019

Conversation

Aulud
Copy link

@Aulud Aulud commented Nov 10, 2019

Add bugfixes and minor features to View command UI

Summary: Fixes a few critical issues with the View command UI and introduces several minor features to improve UX. Verified to work (without severe bugs) on both Windows 10 and Ubuntu 16.04 LTS operating systems.

Changelog

  • Fix total activity spending including the amount from deleted expenses and settlements
  • Fix soft-deleted expenses not appearing visually different from other expenses
    • Soft-deleted expenses and settlements will now have all their text and participant tags greyed out
  • Display the index of each expense in an activity in its description
  • Add CSS styling to visually distinguish settlements from expenses, even when soft-deleted
  • Contract amounts less than $0.01 to <$0.01 when displayed in PersonDetailsPanel, ExpenseCard and TransferCard
  • Display the placeholder message no email provided in place of a contact's email when viewing a contact with no specified email
  • Refactor JavaFX UI classes
    • Add AssertionError detail messages to existing Java assertions
    • Correct unnecessary public access modifier for instance attributes
    • Extract business logic from ActivityCard and ActivityDetailsPanel, replacing them with utility methods exposed by the Activity class
    • Extract common utility methods for string formatting into the utility class UiUtil
    • Collate all conditional (dynamic) CSS styling rules into Extensions.css
  • Fix incorrect format specifier in MESSAGE_SUCCESS of SettleCommand
  • Fix mistakes in the getSolution method exposed by the Activity class

Last updated 11 Nov 2019, 01:30AM

@Aulud Aulud added the type.Task Something that needs to be done, but not a story, bug, or an epic. label Nov 10, 2019
@Aulud Aulud added this to the v1.4 milestone Nov 10, 2019
@Aulud Aulud requested a review from a team November 10, 2019 17:10
@Aulud Aulud added the type.Enhancement New feature or request label Nov 10, 2019
Copy link

@daekoon daekoon left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type.Enhancement New feature or request type.Task Something that needs to be done, but not a story, bug, or an epic.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants