WebMVC - Cosmetic Usability and Display Issues.

[hetal-kapadia]

I am getting accustomed to micro-services and containers and while exploring this solution, noticed a few usability/display issues (noticeable in Firefox) so thought of documenting here.

I am also attempting to get my feet wet with open source contributions so will look to fix these as well.

Here are the issue details
In Cart View and Order Details view:

1. Total is displaying without 2 decimal places.
2. User hover menu is overlapping with page contents, and menu closes before user could move to cursor to Log Out option.

Please excuse and guide me if i am not following the contribution process correctly.

Thank you!

[sughosneo]

Hi @hetal-kapadia , thank for your pointing it out. Will review the proposed changes.
I have linked the issue against the PR #1582

[hetal-kapadia]

Thank you @sughosneo! I am also looking for some guidance to see if i am following the process right.

Since i am new to this process, i see that there were 2 issues created for the same item. #1581 was created explicitly by me to document the issue. #1582 got created when i raised a PR for this fix.

Since i was fixing this issue, should i have waited and documented this while raising the PR to avoid duplication?

[sughosneo]

Hi @hetal-kapadia, you have followed the right process 👍
It's always good to create an issue first and then raise a PR against that issue. In both scenarios, it creates a separate issue.

Hope this helps !