CHANGELOG.md

@@ -1,15 +1,3 @@
-# [0.1.0-develop.10](https://git.lumeweb.com/LumeWeb/sdk/compare/v0.1.0-develop.9...v0.1.0-develop.10) (2023-10-10)
-
-
-### Bug Fixes
-
-* improve components ([f9664fc](https://git.lumeweb.com/LumeWeb/sdk/commit/f9664fc8498ec94e4a355f2e920bd7f706f59145))
-
-
-### Features
-
-* allow to customize the lume identity trigger ([099b2f2](https://git.lumeweb.com/LumeWeb/sdk/commit/099b2f2c649d47df83c060f3cd587ea2c8034060))
-
 # [0.1.0-develop.9](https://git.lumeweb.com/LumeWeb/sdk/compare/v0.1.0-develop.8...v0.1.0-develop.9) (2023-10-10)
 
 

npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
     "name": "@lumeweb/sdk",
-    "version": "0.1.0-develop.10",
+    "version": "0.1.0-develop.9",
     "lockfileVersion": 3,
     "requires": true,
     "packages": {
         "": {
             "name": "@lumeweb/sdk",
-            "version": "0.1.0-develop.10",
+            "version": "0.1.0-develop.9",
             "dependencies": {
                 "@lumeweb/kernel-network-registry-client": "0.1.0-develop.10",
                 "@lumeweb/libkernel": "0.1.0-develop.65",

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@lumeweb/sdk",
-    "version": "0.1.0-develop.10",
+    "version": "0.1.0-develop.9",
     "type": "module",
     "main": "lib/index.js",
     "types": "lib/index.d.ts",

src/components/lume/LumeDashboard/LumeDashboard.tsx

@@ -4,7 +4,6 @@ import Logo from "../../../assets/lume-logo.png";
 import { cva } from "class-variance-authority";
 import { cn } from "../../utils";
 import { useState, useEffect } from "react";
-import React from "react";
 
 const SYNCSTATE_TO_TEXT: Record<Network["syncState"], string> = {
   done: "Synced",
@@ -12,11 +11,8 @@ const SYNCSTATE_TO_TEXT: Record<Network["syncState"], string> = {
   syncing: "Syncing",
 };
 
-export const LumeDashboardTrigger = Dialog.Trigger;
-LumeDashboardTrigger.displayName = "LumeDashboardTrigger";
-
-const LumeDashboard = ({children}: React.PropsWithChildren) => {
-  const { lume: { networks } } = useLume();
+const LumeDashboard = () => {
+  const { networks } = useLume();
 
   const [uniqueNetworkTypes, setUniqueNetworkTypes] = useState<string[]>([]);
 
@@ -26,29 +22,9 @@ const LumeDashboard = ({children}: React.PropsWithChildren) => {
     setUniqueNetworkTypes(uniqueTypes);
   }, [networks]);
 
-  const DefaultTrigger = () => (
-    <LumeDashboardTrigger asChild>
-      <button className="bg-primary text-primary-foreground p-2 px-4 text-sm font-semibold font-mono rounded-md">
-        Open Dashboard
-      </button>
-    </LumeDashboardTrigger>
-  );
-  const GivenTrigger = React.Children.toArray(children)
-    .filter((c) => {
-      console.log({component: c})
-      if(typeof c === 'object'){
-        //@ts-expect-error -- I dont know what the type of this should be, i just know that this works
-        return c.type?.displayName === "LumeDashboardTrigger"
-      }
-
-      return false
-    })
-    .at(0);
-  const Trigger = GivenTrigger ? () => GivenTrigger : DefaultTrigger;
-
   return (
     <Dialog.Root>
-      <Trigger />
+      <Dialog.Trigger>Open</Dialog.Trigger>
       <Dialog.Portal>
         <Dialog.Overlay className="fixed z-40 inset-0 bg-black bg-opacity-50 backdrop-blur-sm" />
         <Dialog.Content className="fixed p-5 z-50 right-0 bottom-0 top-0 w-[300px] bg-neutral-950 text-white border-black border-l data-[state=closed]:slide-out-to-right data-[state=open]:slide-in-from-right ease-in-out data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:duration-300 data-[state=open]:duration-500">

src/components/lume/LumeIdentity/LumeIdentity.tsx

@@ -1,4 +1,3 @@
-import React, { type FC } from "react";
 import { Button } from "../../ui/button";
 import {
   SwitchableComponent,
@@ -9,6 +8,7 @@ import * as ComponentList from "./components";
 import { LazyMotion, domAnimation } from "framer-motion";
 import * as Dialog from "@radix-ui/react-dialog";
 import LumeLogoBg from "./LumeLogoBg";
+import type { FC } from "react";
 
 const LumeIdentity: FC = () => {
   const { visibleComponent, setVisibleComponent } = useSwitchableComponent(
@@ -106,31 +106,14 @@ const LumeIdentity: FC = () => {
 // contrast is not very good, as im testing on a train with a lot of sunlight
 // hitting my screen, it is almost impossible to see whats happening the outline
 // buttons have no contrast
-export const LumeIdentityTrigger = Dialog.Trigger;
-LumeIdentityTrigger.displayName = "LumeIdentityTrigger";
-export default function Wrapped({ children }: React.PropsWithChildren) {
-  const DefaultTrigger = () => (
-    <LumeIdentityTrigger asChild>
-      <button className="bg-primary text-primary-foreground p-2 px-4 text-sm font-semibold font-mono rounded-md">
-        Open Lume
-      </button>
-    </LumeIdentityTrigger>
-  );
-  const GivenTrigger = React.Children.toArray(children)
-    .filter((c) => {
-      if(typeof c === 'object'){
-        //@ts-expect-error -- I dont know what the type of this should be, i just know that this works
-        return c.type?.displayName === "LumeIdentityTrigger"
-      }
-
-      return false
-    })
-    .at(0);
-  const Trigger = GivenTrigger ? () => GivenTrigger : DefaultTrigger;
-
+export default function Wrapped() {
   return (
     <Dialog.Root>
-      <Trigger />
+      <Dialog.Trigger asChild>
+        <button className="bg-primary text-primary-foreground p-2 px-4 text-sm font-semibold font-mono rounded-md">
+          Open Lume
+        </button>
+      </Dialog.Trigger>
       <Dialog.Portal>
         <Dialog.Overlay className="fixed z-40 inset-0 bg-black bg-opacity-50 backdrop-blur-sm" />
         {/* @ditorodev: `left-[calc(50%-192px)] top-[calc(50vh-174px)]` these two are me being dumb and lazy, would be cool to fix with proper centering */}

src/components/lume/LumeIdentity/LumeIdentityContext.ts

@@ -1,23 +1,23 @@
+import { useState } from "react";
 import {
   login,
   // loginComplete,
   // logoutComplete,
 } from "@lumeweb/libkernel/kernel";
-import { useLume } from "../LumeProvider";
 
 export function useLumeIndentity() {
-  const {isLoggedIn, setIsLoggedIn} = useLume();
+  const [loggedIn, setLoggedIn] = useState(false);
 
   return {
-    isSignedIn: isLoggedIn,
+    isSignedIn: loggedIn,
     async signIn(key: Uint8Array) {
       await login(key);
       // await loginComplete(); # this function is buggy `auth.ts:42 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'promise') `
-      setIsLoggedIn(true);
+      setLoggedIn(true);
     },
     async signOut() {
       // await logoutComplete();
-      setIsLoggedIn(false);
+      setLoggedIn(false);
     },
   };
 }

src/components/lume/LumeProvider.tsx

@@ -30,15 +30,13 @@ type LumeObject = {
   networks: Network[];
 };
 
-type LumeContextType = {
-  isLoggedIn: boolean,
-  setIsLoggedIn: (value: boolean) => void,
+type LumeContext = {
   lume: LumeObject;
 };
 
 const networkRegistry = createNetworkRegistryClient();
 
-const LumeContext = createContext<LumeContextType | undefined>(undefined);
+const LumeContext = createContext<LumeContext | undefined>(undefined);
 
 const LumeProvider = ({ children }: { children: ReactNode }) => {
   const [lume, setLume] = useState<LumeObject>({ networks: [] });
@@ -123,10 +121,9 @@ const LumeProvider = ({ children }: { children: ReactNode }) => {
       statusUnsubs.current.forEach((unsub) => unsub());
     };
   }, []);
-  const [isLoggedIn, setIsLoggedIn] = useState(false);
 
   return (
-    <LumeContext.Provider value={{ lume, isLoggedIn, setIsLoggedIn }}>{children}</LumeContext.Provider>
+    <LumeContext.Provider value={{ lume }}>{children}</LumeContext.Provider>
   );
 };
 
@@ -139,5 +136,6 @@ export function useLume() {
     throw new Error("useLume must be used within a LumeProvider");
   }
 
-  return ctx;
+  const { lume } = ctx;
+  return lume;
 }

src/index.ts

@@ -2,6 +2,6 @@ export {
   default as LumeProvider,
   useLume,
 } from "./components/lume/LumeProvider";
-export { default as LumeDashboard, LumeDashboardTrigger } from "./components/lume/LumeDashboard/LumeDashboard";
-export { default as LumeIdentity, LumeIdentityTrigger } from "./components/lume/LumeIdentity/LumeIdentity";
+export { default as LumeDashboard } from "./components/lume/LumeDashboard/LumeDashboard";
+export { default as LumeIdentity } from "./components/lume/LumeIdentity/LumeIdentity";
 import "../styles/globals.css";