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]
