Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot edit button label in Safari #77

Closed
okonet opened this issue Jun 21, 2022 · 9 comments
Closed

Cannot edit button label in Safari #77

okonet opened this issue Jun 21, 2022 · 9 comments
Labels
area:canvas Anything related to rendering on canvas complexity:medium Up to 1 week of work prio:2 Always look for prio:1 issues first before working on prio:2 type:bug Something isn't working

Comments

@okonet
Copy link
Contributor

okonet commented Jun 21, 2022

To reproduce

  1. open https://alpha.webstudio.is/designer/794f3f9a-5a3f-4d65-9006-9e7946a1ff68
  2. Double-click it and try to edit the label

I cannot edit the label after the first keystroke.

CleanShot.2022-06-21.at.11.01.41.mp4

Also, sometimes the cursor is being shown in a wrong position.

@kof
Copy link
Member

kof commented Jun 21, 2022

I think we should focus on Chrome for now and push everything else for later stages. We need to inform the user to prevent them see broken experience, wdyt?

If agree we need an issue to add browser detection and a banner, preventing user from using the designer. Published site or even preview will still work perfectly fine, so its really just for the designer

@kof kof added type:bug Something isn't working complexity:medium Up to 1 week of work area:canvas Anything related to rendering on canvas labels Jun 21, 2022
@okonet
Copy link
Contributor Author

okonet commented Jun 21, 2022

Frankly I think it’s a slippery road to go since I can’t imagine eliminating that kind of technical debt later on. Or do you have a good plan for it?

@kof
Copy link
Member

kof commented Jun 21, 2022

it’s a slippery road to go

I don't love it, but the fact is we would have to test each change for each feature in each browser, right now, which effectively would cost us valuable time, especially when releasing every feature.

For example webflow still only supports chrome and safari, here is the overlay they show in Firefox:

Screenshot 2022-06-21 at 15 43 58

To make it easier to support more browsers later, I think we need to start writing down the list of browser APIs that we already rely on and that won't work in Firefox for example. Then we need to start looking into fallback solutions to those apis if they are possible at all.
Maybe need to start marking code that doesn't work in certain browsers with a tag e.g. @experimental

I just think we want to be on the cutting edge of the APIs and only support browsers which allow us to do so in regards to Designer UI, the site itself should work everywhere ofc.

@kof
Copy link
Member

kof commented Jun 21, 2022

A few more things to keep in mind when deciding to support many browsers for Designer interface:

  1. Designer is a very complex program, not a regular site, it could be generally very hard to support certain interactions in multiple runtimes
  2. Supporting multiple runtimes puts a constraint on you when deciding how to build new things, can't just blindly build on the cutting edge features and release now. Browsers will catch up later eventually, but you will have to wait potentially years for certain API until you can use it, its often not polyfillable or not in a good way
  3. Designer experience itself is not as frequently used as a published site. Asking user to open Chrome or Edge is today a very small ask, I am not sure that's a problem at all, especially considered our users are not complete noobs most of the time.

@okonet
Copy link
Contributor Author

okonet commented Jun 22, 2022

@kof it makes sense from the business standpoint at this moment of time but my concern is accessibility and adoption. The more barrier we put upfront the worse it is for the adoption. Just as an example: I can run Figma on my iPhone and it runs almost exactly as it is on desktop (with viewport constraints etc. of course).

I think it's fine to take shortcuts while chasing for the beta and maximizing the value for the majority of users (yet we don't know what browsers they will be using just yet but Chrome dominates the web either way) but IMO we should mark features / bugs somehow in a consistent way in order to get rid go them later. Your proposal to use JSDoc syntax makes sense so let's settle on this asap and start documenting?

@kof
Copy link
Member

kof commented Jun 22, 2022

Yeah, we need to see if we can at least make it work in safari. Touch devices are a separate challenge though, because all the drag&drop interactions need to work. I really want webstudio to run on a tablet some day, but I am afraid right now it would cost us a lot of time.

@kof
Copy link
Member

kof commented Jun 22, 2022

how do we call it for inline documentation? JSDOC doesn't have a special tag for it, maybe

@unsupported safari, firefox

@okonet
Copy link
Contributor Author

okonet commented Jun 22, 2022

@unsupported safari, firefox

SGTM!

@kof kof added the prio:2 Always look for prio:1 issues first before working on prio:2 label Jul 11, 2022
@kof
Copy link
Member

kof commented Feb 6, 2023

not reproducible any more

@kof kof closed this as completed Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:canvas Anything related to rendering on canvas complexity:medium Up to 1 week of work prio:2 Always look for prio:1 issues first before working on prio:2 type:bug Something isn't working
Projects
No open projects
Status: Done
Development

No branches or pull requests

2 participants