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

Improved display of parking options on property pages (multiParkingLot) #939

Open
4 tasks done
fredericalpers opened this issue Oct 22, 2024 · 9 comments · May be fixed by #943
Open
4 tasks done

Improved display of parking options on property pages (multiParkingLot) #939

fredericalpers opened this issue Oct 22, 2024 · 9 comments · May be fixed by #943
Assignees
Labels
component: property detail page Issue, Pull Request or Discussion related to property detail page QA Issue or Pull request that is in review UI/UX Issue, Pull Request or Discussion related to UI/UX

Comments

@fredericalpers
Copy link
Member

fredericalpers commented Oct 22, 2024

Improved display of parking options on property pages (multiParkingLot)

User Story

Currently, the parking options in the property front-end template are displayed as simple text such as “2 carports at 10000 € each (purchase)”. This display is not very flexible and not user-friendly. Our goal is to create a more individual and user-friendly display of parking options based on actual data and to hide excess information such as prices of €0.

  • Improvement of the user experience through an appealing and comprehensible presentation of the parking options.
  • Reduce unnecessary information to increase the relevance of the display.

Scenario 1 (with prices):
2 Carport, 10,000 € (purchase), 3 Duplex, 300 € (rent)

Scenario 2 (with and without price):
2 carport, 1 garage, € 5,000 (purchase)

Scenario 3 (without price):
2 carport, 1 garage

Task / Requirements

  • If the price of a parking space is 0.00, this price should not be displayed.
  • The marketing type (e.g. “purchase” or “rent”) should also not be displayed in this case.
  • Only if the price is greater than 0.00 should both the price and the marketing type (purchase or rent) be displayed.
  • The presentation of the parking options should be compact and yet informative.
@fredericalpers fredericalpers added UI/UX Issue, Pull Request or Discussion related to UI/UX component: property detail page Issue, Pull Request or Discussion related to property detail page labels Oct 22, 2024
@fredericalpers fredericalpers added this to the v5.9 milestone Oct 22, 2024
@dai-eastgate dai-eastgate self-assigned this Oct 25, 2024
@dai-eastgate
Copy link
Contributor

@fredericalpers Please take a look at my demo video and let me know your opinions. Thanks!

multi-parkinglot.mp4

@fredericalpers
Copy link
Member Author

@dai-eastgate looks good to me! thank you :) I will review this as soon as possible.

@fredericalpers
Copy link
Member Author

fredericalpers commented Oct 29, 2024

@dai-eastgate I do have on more question. Can the prices also be displayed in currencies other than the euro? E.g. a swiss customer with swiss franks (CHF) or even Đồng ? When changing the currency in enterprise to CHF the website frontend still displays Euro

@dai-eastgate
Copy link
Contributor

Can the prices also be displayed in currencies other than the euro?

@fredericalpers Yes, prices can be displayed in currencies other than the euro.

When changing the currency in enterprise to CHF the website frontend still displays Euro

If you are changing to CHF but it still shows euro, this usually happens when the "waehrung" field has not been added to the configuration. Please add it and let me know the results.

@fredericalpers
Copy link
Member Author

@dai-eastgate Thank you for the update! However, from a user perspective, this implementation might not be ideal. Customers generally would not want to see an additional entry with "Currency: XX" in the list. Would it be possible to adjust this so that the configured value in Enterprise is used directly instead? This would create a cleaner experience for users.

@dai-eastgate
Copy link
Contributor

@fredericalpers Are you suggesting that you want the correct currency to display without needing to configure the currency field?

  1. When displaying parking, the currency will not be shown even if it is configured.
  2. If you only configure the currency field without displaying parking, the currency will still be shown.

Please clarify if this is what you meant!

@fredericalpers
Copy link
Member Author

@dai-eastgate Currently I have to add the Currency field to be able to show the correct currency for the MultiParkingLot field. Ideally I do not have to add the extra currency field for the correct display on the MultiParkingLot field.

stellpl1
stellpl2
stellpl3

@dai-eastgate
Copy link
Contributor

@fredericalpers I fixed it. Please take a look at my demo video and let me know your opinions. Thanks!

parking.space.mp4

@fredericalpers
Copy link
Member Author

Thank you, I will review this as soon as possible :)

@fredericalpers fredericalpers added the QA Issue or Pull request that is in review label Nov 5, 2024
@fredericalpers fredericalpers removed this from the v5.9 milestone Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: property detail page Issue, Pull Request or Discussion related to property detail page QA Issue or Pull request that is in review UI/UX Issue, Pull Request or Discussion related to UI/UX
Projects
None yet
2 participants