Feature/visual room browser v4

[odzhychko]

Cleaned up version of #8264
Resolves #8724

Before (main)

After

Screencast.From.2026-08-15.20-01-26.mp4

🤖 AI (if applicable)

• The content of this PR was partly or fully generated using AI

[codecov]

Codecov Report

❌ Patch coverage is 96.18321% with 5 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/models/resourceProps.ts	44.44%	4 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[odzhychko]

@SebastianKrupinski Let me know if you find something with regards to code organization.
@nimishavijay Let me know if you find something with regards to UI/UX.

[SebastianKrupinski]

Okay I tested this this works kind of:

As mentioned already all the fancy spiting and join does not really work and only works for "Netherlands" style address format.

[odzhychko]

Okay I tested this this works kind of:

Your screenshot looks of. Should look like in the description have you checked out feature/visual-room-browser-v4 (-v4)?

[SebastianKrupinski]

Okay I tested this this works kind of:

Your screenshot looks of. Should look like in the description have you checked out feature/visual-room-browser-v4 (-v4)?

Yeah, I think so... but here is another take

[odzhychko]

Okay I tested this this works kind of:

Your screenshot looks of. Should look like in the description have you checked out feature/visual-room-browser-v4 (-v4)?

Yeah, I think so... but here is another take

Looks still wrong...

Seating capacity should be indicate In the bottom left and not top right.

[SebastianKrupinski]

Hey,

Retested this again last night... Found two minor issues...

a) when there are no rooms, we should hide all the search functions

b) I have 2 buildings with floor 1... but they are combined in this drop down in to a single floor... since we now have a building name... maybe display the building name and the floor?

[odzhychko]

a) when there are no rooms, we should hide all the search functions

Fixed. The button is now hidden, if no rooms are available.

[odzhychko]

b) I have 2 buildings with floor 1... but they are combined in this drop down in to a single floor... since we now have a building name... maybe display the building name and the floor?

1. Seems you haven't tested with the changes Feat/add room building name property calendar_resource_management#302 . Because then you would see a filter for "Building".

2. As we have a filter for building (if building names are provided by the resource provider), a filter for floors that includes the building seems odd. So I'd like to leave it as is.

[SebastianKrupinski]

b) I have 2 buildings with floor 1... but they are combined in this drop down in to a single floor... since we now have a building name... maybe display the building name and the floor?

1. Seems you haven't tested with the changes [Feat/add room building name property calendar_resource_management#302](https://github.com/nextcloud/calendar_resource_management/pull/302) . Because then you would see a filter for "Building".

My bad, I did pull the latest version but forgot to run npm ci

Much better now

[SebastianKrupinski]

Lol. I guess you had auto merge turned on. Cause I just squashed and re-based it for you and it now merged on its own.