EnxDev commented on PR #32112:
URL: https://github.com/apache/superset/pull/32112#issuecomment-2726923421

   > Thank yo so much @EnxDev ❤️
   
   @kasiazjc Thanks for your review as well ❤️
   > 
   > A few things I found so far
   > 
   > * Action icons in CRUD views are 24x24px and should be 16x16px for a more 
balanced look (for example favorites star is 16x16) DONE ✅
   > 
   > <img alt="image" width="1293" 
src="https://private-user-images.githubusercontent.com/36897697/420383908-985f6da1-ce01-4f2b-a350-a28d8bfaa18f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODM5MDgtOTg1ZjZkYTEtY2UwMS00ZjJiLWEzNTAtYTI4ZDhiZmFhMThmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQyYmVmMDkzMTI5OGE0MmM1M2VlYmJkOTk5YjE0NzlhNmEwYTNkMWZmYmMwMzVmZjMxNTFjZGI4Yjg1MTVlMTcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.qD7R83LrugnhDZdIYMbsArswZCqldAi1NCORiqGHVcc";>
   > * Icons in buttons should be bigger (10px is not really visible), I can 
see in ant-d they are 14x14px and we can follow that for keeping our 
implementation pretty much vanilla without lots of deviations. I can see that 
on master we use in here some weird size like 10x12, but the previous icon was 
much more bold, so it needs to be bigger. NOTE: I found that in some buttons we 
use 16px (for example save button in explore), so not sure what the rule is. Is 
this the correct time to update it? @EnxDev @geido
   DONE ✅
   > 
   > <img alt="image" width="373" 
src="https://private-user-images.githubusercontent.com/36897697/420385184-84a96c95-8fb1-4628-8d57-0ca9c8a57d54.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODUxODQtODRhOTZjOTUtOGZiMS00NjI4LThkNTctMGNhOWM4YTU3ZDU0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWIwMWYxYWY0OWJmMDdhZjNjZjUyNGNmOTRkNDZlNjMyYTEwZDJiNGM5NzZlNDI0ZjUxNDNjMzhjMTI2ZjI0ZTUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.aUyibfyXtZJoX2sGqpfURz2cOG-wP3MloBXf5_zCGsg";>
   > * might be old implementation, but this icon should match other button 
icons like mentioned above (14px)
   DONE ✅
   > 
   > <img alt="image" width="335" 
src="https://private-user-images.githubusercontent.com/36897697/420394656-15aaac66-3428-4975-b396-b039a7359f9d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzOTQ2NTYtMTVhYWFjNjYtMzQyOC00OTc1LWIzOTYtYjAzOWE3MzU5ZjlkLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdlZTFlODA3MWRjOTBhMWRmZmQwNzk0NGFkMWNmMzk3OTRmN2RlNWJjNTgyMTBlNmVlOGEzMDcxMDU4MDMyOTEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.1owdJANzDgKbC6xFXfC5EAfhBx70wlhf_DxOxb5KPng";>
   > * For somer reasons in the navbar spacings between caret-down and 
text/icons disappeared. Can you add at least 4px spacings?
   DONE ✅
   > 
   > now: <img alt="image" width="949" 
src="https://private-user-images.githubusercontent.com/36897697/420386914-5800be7a-281c-4c73-b88a-2a205f7d818c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODY5MTQtNTgwMGJlN2EtMjgxYy00YzczLWI4OGEtMmEyMDVmN2Q4MThjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTE4NjMyYjkwM2FjMTgzZTMwZGFmNjRiZWRmNDMxZDY0NzJiYzZkODQwYzNiNDhlOTk4M2EyZDQ1ODExYmYzMjQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.WH-3IM2ymO3MRawd2sbZU4Kl6i5hOgsnnK9ERp9sgPc";>
   > 
   > before: <img alt="image" width="949" 
src="https://private-user-images.githubusercontent.com/36897697/420387026-9a58d041-6e0c-416b-af95-2ce7f301bb1e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODcwMjYtOWE1OGQwNDEtNmUwYy00MTZiLWFmOTUtMmNlN2YzMDFiYjFlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTEwYjljNDNjYjY2NTA1YjQ2N2VhNjZjYjUxZTk2NjBjMzFmZmE5M2I2NjA0ZjVmZTI3NWZhOTQyNGRiMGQ3ZmYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.zpDi6kNdAfjKVt4SXfRBzIDz1lu4gqG80W4uag8Jayc";>
   > 
   > * Controls in control panel - in the new implementation the caret-right is 
filled, and should be just an empty arrow like currently
   DONE ✅
   > 
   > now: <img alt="image" width="302" 
src="https://private-user-images.githubusercontent.com/36897697/420387574-ccb0b01f-d5e8-4b51-b394-32fc9e728c80.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODc1NzQtY2NiMGIwMWYtZDVlOC00YjUxLWIzOTQtMzJmYzllNzI4YzgwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWM0NmZmZWRhMDk1Y2E1ZWEzZjFmYzZjYzdiNWFkMzFjZGJmNGNhMzM5MmFiNTI3ZTM0NmMzZjAwM2NiNjExZjEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.qlKkolRgiW8PYW94PL6zVJDOEQN6jUbglSIxHmOClc4";>
   > 
   > before: <img alt="image" width="302" 
src="https://private-user-images.githubusercontent.com/36897697/420387762-93d1c7d9-06b7-45be-94a6-adc253014af9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODc3NjItOTNkMWM3ZDktMDZiNy00NWJlLTk0YTYtYWRjMjUzMDE0YWY5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTljNmQyMmQ0NzYzZWRkZGFiZWQ3YzY0MWM5ZTU0N2VjODE5YTNmNzE4ZWIwZDdlNTcxNTc2MGEzZWI0YTMxYTYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.GdJVAyeplTGUGa8ZPTWStFrXsywC0Fqq4ZZyZkzZyXA";>
   > 
   > * in dashboard filters panel (vertical) sizing between gear and collapse 
icon got bigger than before. It should be 8px DONE ✅
   > 
   > now: <img alt="image" width="258" 
src="https://private-user-images.githubusercontent.com/36897697/420389494-46a6ac5b-411f-427c-b791-2b33a91af1e6.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODk0OTQtNDZhNmFjNWItNDExZi00MjdjLWI3OTEtMmIzM2E5MWFmMWU2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU4ODQzMWQ2ODVhNmNlYzc1YzMzNTFkODA4ZWJlZjYyZTM4OTUzNGNlYmRiOTg5ZTE4ODQ5MzcxZDUyODM0ZWImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.2uesDwh0PsEzyfjxEWE4i5DgdUmOUAWaTFvEC_qnYGQ";>
   > 
   > before: <img alt="image" width="258" 
src="https://private-user-images.githubusercontent.com/36897697/420389344-d8166a20-efec-4c68-81c0-88a7e811f80b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzODkzNDQtZDgxNjZhMjAtZWZlYy00YzY4LTgxYzAtODhhN2U4MTFmODBiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ0ODcyNDFhYjllZjRhNTY3MDZjYjFkMjQ1MjI5YzAyYjVkMWUwOWRiMGQ5ZTFhMWZjYmYzZDQ3OTA2ZjhlZjYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.5w2UUFk77Ou5TRPn6XDNY_v0HKdzZA4JCol8QORGu7w";>
   > 
   > * Icon in dataset settings should be 16px for better balance DONE ✅
   > 
   > <img alt="image" width="972" 
src="https://private-user-images.githubusercontent.com/36897697/420392649-d9a6484b-4615-46da-a15d-aa7ab7802354.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzOTI2NDktZDlhNjQ4NGItNDYxNS00NmRhLWExNWQtYWE3YWI3ODAyMzU0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWQ2OTRlMDA1ZTdlYTlkZTI4OGZlNmRmNDcyNGJkOGNjMjhmNWMwZGFjOTAwODk3MzM3YmExMzczNjcxZjRjNTcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.aNTeI3WiBK1zZMNokbNQNtv_joK-NgdHh5ZM2P-seJM";>
   > * not sure if this icon should be new too, but pencil doesnt seem to be 
updated to our current edit icon (like above) DONE ✅
   > 
   > <img alt="image" width="972" 
src="https://private-user-images.githubusercontent.com/36897697/420392981-e03b3236-0e43-465a-9d74-2e11bb41f2cc.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzOTI5ODEtZTAzYjMyMzYtMGU0My00NjVhLTlkNzQtMmUxMWJiNDFmMmNjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTcwZjljMGYxY2RkYTVmZDQwMTg5Mzk2MGY4Y2U0ZjhhMWEyMjc2NzcyNjJmMThiMzMwMzAyNzc4ODE3M2M3NjkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.65XPV0uOlwzrxDj328bpQPrkK_b9Mu5rpDHlTkMd6EA";>
   > * this caret (collapse/expand) icon on the right should match icons on the 
left (arrow not filled) DONE ✅
   > 
   > <img alt="image" width="725" 
src="https://private-user-images.githubusercontent.com/36897697/420393612-2cc67b78-1250-40e1-b3c2-35bdc3cc08ca.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzOTM2MTItMmNjNjdiNzgtMTI1MC00MGUxLWIzYzItMzViZGMzY2MwOGNhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTM3MWJlMjM3Zjc0MWViMDg4OTQ0OGM1ODdkZmFkN2IxNDk3YTJiMmVkYzg4N2YzMDE1NTg1MDIzMWI3ZjE4MjImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.bmnc0ixxXSHbt-AIePkiWICrBHQQcWab4I4RSmBJUOA";>
 <img alt="image" width="972" 
src="https://private-user-images.githubusercontent.com/36897697/420393454-3cd964db-11e8-47ff-b506-98791b0eb66d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6Ik
 
pXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIwNTUyMTksIm5iZiI6MTc0MjA1NDkxOSwicGF0aCI6Ii8zNjg5NzY5Ny80MjAzOTM0NTQtM2NkOTY0ZGItMTFlOC00N2ZmLWI1MDYtOTg3OTFiMGViNjZkLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAzMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMzE1VDE2MDgzOVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWIzNWRmYWIyZTNiY2E2Y2NjM2RjMTgzYmVkMzg3ZWExYjhjOThmOTYxODUzOTU2YjVkNDdmZjY0ZTY1NzEwMzkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.HcnnvYmf3bCLQ11ds6-27y3JXxA0Pvt38wNlCey1zws">
   
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to