denoland / dotland

[Archived] deno.land website
https://deno.land
MIT License
957 stars 626 forks source link

refactor: redesign add module page #2252

Closed crowlKats closed 2 years ago

bartlomieju commented 2 years ago

This error is visible immediately after visiting the page

Screenshot 2022-07-20 at 21 50 04
kitsonk commented 2 years ago

Sorry for the late feedback, but it feels odd that this is copyable/visible until there is a valid package name:

Third_Party_Modules___Deno

Also, if you type a valid package name (e.g. oak_se) and then you type a taken valid name (e.g. oak_sessions) the confetti goes off and the UI updates for success:

Third_Party_Modules___Deno
crowlKats commented 2 years ago

@kitsonk fixed, PTAL

kitsonk commented 2 years ago

Maybe I misunderstood. I am still getting "success" and confetti when going from a valid name to an invalid name:

Third_Party_Modules___Deno
crowlKats commented 2 years ago

huh, i cannot reproduce that anymore with the latest deployment (https://dotland-m8jyb21w6dwg.deno.dev)

kitsonk commented 2 years ago

https://user-images.githubusercontent.com/1282577/180760202-1ca3c74d-288b-4c6d-aca3-1611776e50d3.mp4

crowlKats commented 2 years ago

ah, after spamming delete and writing name fast, i could reproduce again...

kitsonk commented 2 years ago

In my video I wouldn't consider it fast typing. 😜

crowlKats commented 2 years ago

ok i really cannot reproduce the way you are doing it...

rojvv commented 2 years ago

@crowlKats you can try this with the name twitter, too

crowlKats commented 2 years ago

ok, so it seems you need to be typing at a specific speed and then it happens; love it.

kitsonk commented 2 years ago

ok i really cannot reproduce the way you are doing it...

"works on my machine"... 🤣 Version 1.40.113 Chromium: 103.0.5060.114 (Official Build) (x86_64)

kitsonk commented 2 years ago

On Firefox 103, I have to type really fast, but it breaks in a different way, no "error" that module is taken and sort of infinite confetti that looks really yucky:

Third_Party_Modules___Deno_and_refactor__redesign_add_module_page_by_crowlKats_·_Pull_Request__2252_·_denoland_dotland
crowlKats commented 2 years ago

this is extremely annoying... how about now?

kitsonk commented 2 years ago

Same behaviour for me on: https://dotland-2m9t6qw1bv5g.deno.dev/add_module

crowlKats commented 2 years ago

@kitsonk check again

kitsonk commented 2 years ago

Ok, the confetti has gone and works as expected, thought my manual fuzzing has demonstrated another minor issue. Long package names cause layout problems:

Third_Party_Modules___Deno

crowlKats commented 2 years ago

oh, thats fine, thats simple to fix, thank god the false-positive is fixed now