[Beta] Minor issues/comments #340
-
Beta is looking great! I noticed a few things outlined below:
|
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 1 reply
-
Thanks! |
Beta Was this translation helpful? Give feedback.
-
Ahh...got it. 👍 The only other thing I noticed is that the app appears to be making a check license request to AWS on every load here. You might be able to save a bunch of AWS calls by checking if the license exists in storage before making that call...just a thought. |
Beta Was this translation helpful? Give feedback.
-
Ahh yes thank you! Actually I need to pull that out completely anyway. I had to make my own way to gift licenses to users, but after building that out the payment provider added the option in so I don't need that implementation after all :) So thanks for the reminder of it! |
Beta Was this translation helpful? Give feedback.
-
Another thing I noticed is the text below will appear for a split second occasionally on load. I have noticed the beta version does seem to be loading a little slower than 3.x during a "cold-start" of Chrome...for me at least. |
Beta Was this translation helpful? Give feedback.
-
Hey, just letting you know I've resolved everything mentioned here now. There won't be an update to the beta though, I think I'll be pushing to production in the next couple days. Thank you for your feedback! In regards to the weather, I re-wrote the layouts for the widget. There are now the original two layouts available again, mostly as they were, and the other layouts have been adjusted. |
Beta Was this translation helpful? Give feedback.
Good catch. I did have a reason for that originally, but since I coded that things were adjusted a bit and I don't think my reason makes sense anymore, so I'll fix that! thanks!
The IDs are on purpose. While developing I was noticing myself get confused when switching between different widgets of the same type. So to help users, by default the widget is named as the type followed by it's ID. You are given the option of renaming that widget using the text caret icon to the right. I wasn't sure what another option to help differentiate this would be. Just automatically going "1, 2, etc...) can get complicated in the code due to reordering/adding/removing widgets. If you have another ide…