fix: events and types

Author: jpoehnelt

README.md

@@ -100,10 +100,10 @@ Note: The Google Picker API does not support file organization, moving, or copyi
 
 ### Methods
 
-| Method                | Type                                                                                                                                                 |
-| --------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------- |
-| `addEventListener`    | `(type: "cancel" \| "picked", listener: (ev: CustomEvent<ResponseObject>): void, options?: boolean \| AddEventListenerOptions \| undefined) => void` |
-| `removeEventListener` | `(type: "cancel" \| "picked", listener: (ev: CustomEvent<ResponseObject>): void, options?: boolean \| EventListenerOptions \| undefined) => void`    |
+| Method                | Type                                                                                                                                                  |
+| --------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `addEventListener`    | `(type: "cancel" \| "picked" \| "loaded", listener: (ev: DrivePickerEvent): void, options?: boolean \| AddEventListenerOptions \| undefined) => void` |
+| `removeEventListener` | `(type: "cancel" \| "picked" \| "loaded", listener: (ev: DrivePickerEvent): void, options?: boolean \| EventListenerOptions \| undefined) => void`    |
 
 ### Events
 

src/drive-picker/drive-picker-element.ts

@@ -44,15 +44,19 @@ export interface DrivePickerElementProps {
 	width?: number;
 }
 
+export type DrivePickerEvent = CustomEvent<google.picker.ResponseObject>;
+
+// TODO fix typings for Action to include "loaded"
+
 export interface DrivePickerElementEventListeners {
 	addEventListener(
-		type: "cancel" | "picked",
-		listener: (ev: CustomEvent<google.picker.ResponseObject>) => void,
+		type: "cancel" | "picked" | "loaded",
+		listener: (ev: DrivePickerEvent) => void,
 		options?: boolean | AddEventListenerOptions,
 	): void;
 	removeEventListener(
-		type: "cancel" | "picked",
-		listener: (ev: CustomEvent<google.picker.ResponseObject>) => void,
+		type: "cancel" | "picked" | "loaded",
+		listener: (ev: DrivePickerEvent) => void,
 		options?: boolean | EventListenerOptions,
 	): void;
 }
@@ -275,6 +279,10 @@ export class DrivePickerElement
 
 	private callbackToDispatchEvent(data: google.picker.ResponseObject) {
 		this.visible = Boolean(this.picker?.isVisible());
+		// TODO file bug for this
+		if (data.action === google.picker.Action.CANCEL) {
+			this.visible = false;
+		}
 		this.dispatch(data.action, data);
 	}
 

src/drive-picker/index.ts

@@ -16,6 +16,7 @@
 
 export {
 	DrivePickerElement,
+	type DrivePickerEvent,
 	type DrivePickerElementProps,
 } from "./drive-picker-element";
 

src/utils.ts

@@ -36,7 +36,7 @@ export async function retrieveAccessToken(
 	}
 
 	return new Promise((resolve) => {
-		const client = google.accounts.oauth2.initTokenClient({
+		const client = window.google.accounts.oauth2.initTokenClient({
 			client_id: clientId,
 			scope,