graceguo-supercat edited a comment on pull request #11781: URL: https://github.com/apache/incubator-superset/pull/11781#issuecomment-733920959
@ktmud and @Steejay Your design looks good, but please note only `Source` tab has ReadOnly control. Both header and footer section, are shared with other tabs. We can do hide and show for Padlock if user is in other tab, but I just want to emphasis **"Source" tab** need extra careful protection. I do not want to share Padlock with other tabs with intention. Mostly cases, when user edit Metrics or Columns, they can find error and recover it right away. (Remove existed metrics and columns are dangerous too, you might remove something other ppl use). Edit Metrics and Calculated Columns also happens pretty often, so we don't want to add extra click for this operation. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
