-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[Feature]: Support a way to reference the widget user needs to drag in the deprecation message #14380
Comments
@jnikhila for the message for deprecation widgets, we will need a message along the lines mentioned in the description above. |
@rahulramesha - The message looks good, just changed it a bit:
CC: @dilippitchika |
@jnikhila Thanks for the updated message, I have made a slight modification by removing, @dilippitchika @somangshu I have added a way to reference the replacement widget in the message and updated the message, I have also added a placeholder for a link in the code. Will need to just configure the link once the doc is ready. Please let me know if this requires any changes. |
Thanks @rahulramesha this works, @jnikhila just extending the discussion we had for - #14462 We need a document which explains the following things, please discuss with rahul as well once.
|
@dilippitchika - understood. I'll schedule a call to understand the doc needs with @rahulramesha and yourself and take it ahead. @rahulramesha - for the message, we should keep the heading as before and just modify the old message with the new message as below.
Just a suggestion - I'm not sure if we can do some UI highlighting for the old widget type that is deprecated and the new widget type with which the user has to replace the old widget. It'll quickly grab the user's attention. As in the current message, it looks like plain text without having an emphasis on the types of widgets that I as a user had to replace. Something like this: Heading - Stay Up-to-date CC: @dilippitchika - what do you think? |
@jnikhila, we can have a heading, But from what the users are mentioning on discord. The Heading |
@rahulramesha -
Based on our discussion, let's not keep a heading for now. |
Verified all the widgets placed in Deprecation application Working fine, moving to verified. |
Is there an existing issue for this?
Summary
Within the deprecation message, we wish to inform the user that they need to drag a specific widget to upgrade this.
App viewers pod needs a way to provide the name of the widget the user needs to update to.
The message can be the following -
The widget you are currently using is now deprecated. You can drag a new button widget and configure it to work in place of this widget.
But also check with @jnikhila to finalise.
Qualities of a good message
Why should this be worked on?
This will help us communicate to the user that they must drag the specific widget which maybe sometimes different than the current widget they are working on.
Example
The text was updated successfully, but these errors were encountered: