light9 Changeset - 3f0b889c8fa9

Changeset - 3f0b889c8fa9

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

drewp@bigasterisk.com - 20 months ago 2023-05-31 20:50:13
drewp@bigasterisk.com

don't let phone users click on song uris by mistake

3 files changed with 4 insertions and 3 deletions:

light9/ascoltami/Light9AscoltamiUi.ts

light9/ascoltami/index.html

light9/web/ResourceDisplay.ts

0 comments (0 inline, 0 general)

light9/ascoltami/Light9AscoltamiUi.ts

➞

Show inline comments

@@ @@ -123,13 +123,13 @@ export class Light9AscoltamiUi extends L @@
                 (song) => html`
                   <tr
                     class="row ${classMap({
                       playing: !!(this.song && song.equals(this.song)),
                     })}"
+                  >
                     <td><resource-display .uri=${song}></resource-display></td>
+                    <td><resource-display .uri=${song} noclick></resource-display></td>
                     <td>
                       <button @click=${this.onSelectSong.bind(this, song)}>
                         <span>Select</span>
                       </button>
                     </td>
                   </tr>

light9/ascoltami/index.html

➞

Show inline comments

@@ @@ -39,13 +39,13 @@ @@
       light9-ascoltami-ui {
         flex: 1 1 auto;
+      }
     </style>
     <meta
       name="viewport"
-      content="user-scalable=no, width=500, initial-scale=1"
+      content="user-scalable=no, width=device-width, initial-scale=.7"
     />
     <script type="module" src="../ascoltami/Light9AscoltamiUi"></script>
   </head>
   <body>
     <h1>ascoltami on {{host}}</h1>
     <div id="page">

light9/web/ResourceDisplay.ts

➞

Show inline comments

@@ @@ -74,12 +74,13 @@ export class ResourceDisplay extends Lit @@
   @state() label: string = "";
   @state() renameDialogOpen = false;
   @state() renameTo = "";
   @property({ type: Boolean }) rename: boolean = false;
   @property({ type: Boolean }) noclick: boolean = false;
   @property({ type: Boolean }) minor: boolean = false;
   constructor() {
     super();
     getTopGraph().then((g) => {
       this.graph = g;
@@ @@ -109,13 +110,13 @@ export class ResourceDisplay extends Lit @@
     } else {
       this.label = this.graph.labelOrTail(this.uri);
+    }
+  }
   private href(): string {
     if (!this.uri) {
+    if (!this.uri || this.noclick) {
       return "javascript:;";
+    }
     return this.uri.value;
+  }
   private resClasses() {

0 comments (0 inline, 0 general)