diff --git a/addons/sale/views/sale_order_views.xml b/addons/sale/views/sale_order_views.xml
index 6f68daf36bd..07de02fbe58 100644
--- a/addons/sale/views/sale_order_views.xml
+++ b/addons/sale/views/sale_order_views.xml
@@ -271,7 +271,7 @@
                 <button name="action_unlock" type="object" string="Unlock"
                         invisible="not locked" groups="sales_team.group_sale_manager"/>
                 <button name="action_preview_sale_order" string="Preview" type="object" class="btn-secondary"/>
-                <button name="action_cancel" type="object" string="Cancel" invisible="state not in ['draft', 'sent', 'sale'] or not id" data-hotkey="x"/>
+                <button name="action_cancel" type="object" string="Cancel" invisible="state not in ['draft', 'sent', 'sale'] or not id or locked" data-hotkey="x"/>
                 <button name="action_draft" invisible="state != 'cancel'" type="object" string="Set to Quotation" data-hotkey="w"/>
                 <t groups="sale.group_auto_done_setting">
                     <button name="action_lock" type="object" string="Lock"