elibrumbaugh commented on issue #6964: [WIP][dashboard] direct link to UI 
component in dashboard
URL: 
https://github.com/apache/incubator-superset/pull/6964#issuecomment-469494683
 
 
   Grace and I synced up offline about this today. Documenting the outcome here.
   
   1. Move the link icon from the left of the tab to the right. This will allow 
the user to read the title before link action.
   2. Remove the link icon from chart titles and move it into the chart 
overflow menu with the other chart related actions. The action will be title 
`share chart` This has better parody with the dashboard share interaction.
   
   TBD: Size of the icon. Is it large enough? Most likely wait for user 
feedback on this. Right now it is the same size as the icon in the chart share 
action.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to