Skip to content

Conversation

@willum070
Copy link
Collaborator

@willum070 willum070 commented Dec 10, 2025

This PR adds two new samples to demonstrate AI-powered summaries:

  • ai-powered-summaries-basic is a simplified integration that shows retrieving a generative place summary. Preview: https://jsfiddle.net/dubnewt/o7xhf6cu/
  • ai-powered-summaries is a more full-featured sample that really lets users "kick the tires" of the feature. It's a map with autocomplete that lets users search for places to see the available summaries. Preview: https://jsfiddle.net/dubnewt/5h83bxkg/1/

@snippet-bot
Copy link

snippet-bot bot commented Dec 10, 2025

Here is the summary of changes.

You are about to add 10 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@willum070 willum070 requested a review from ReeseJones December 10, 2025 15:41
@ReeseJones
Copy link

Regarding: Maps JS API Sample: ai-powered-summaries
image

On my rather large screen I was still getting overlap issues with the summary panel. Could anchor to a different side or remove the controls that are not important to the sample?

This "reviews" element seems to be a button? But I am not certain why? Is it doing something?
image

Oh, after looking at the code I see its a tab! Maybe we should leave the tabs all rendered but disable tabs with no content? Although i'm noticing places typically only have 1-2 different summary types... and adding all tabs may be over-crowded.

There is not really any way to tell what places will have what AI summaries, so maybe we should prompt users with some examples?
Hotels in New York typically have neighborhood summaries. (St Regis, hotel 57 etc...), tesla chargers have ev charge ammenity summaries... restaurants and services usually have review summaries etc... Not sure if these samples are in contexts where users would know to try searching these things specifically? Is there a good spot for that (in readme or comments or specific element in demo to prompt?)

Place autocomplete I think by default uses the default color theme of the system:
image
Might be worth forcing it to white to match the rest of the sample?

Regarding: Maps JS API Sample: ai-powered-summaries-basic
Could you make it so clicking the AM re-opens the infowindow?
Do customers have the same legal requirements to show the report summary uri/ai disclosure? (Curious if the link should also be included or not?)

Other Thoughts:
There are a lot of distracting ts-ignore comments. Will those be visible to users looking at the sample? If not can changes to be made to enable removing them? Is that blocked by external typing?

I accidentally updated the wrong file
Disable street view and fullscreen controls on the map.
Updated active tab button styles for better visibility.
* Adds click event for marker to display info window.
* Adds HTML for "report an issue" link.
@willum070
Copy link
Collaborator Author

Thanks so much for reviewing! To answer your question about ts-ignore comments, yes it is 100% due to typings. If typings are up to date then we don't need to use ts-ignore.

@willum070 willum070 requested a review from ReeseJones December 11, 2025 18:02
@willum070 willum070 merged commit 3a1c228 into main Dec 16, 2025
8 checks passed
@willum070 willum070 deleted the ai-powered-summaries branch December 16, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants