-
Notifications
You must be signed in to change notification settings - Fork 59
[$20] - "X" to close Badges pop up is too far to the right #905
Comments
This is launched here - https://www.topcoder.com/direct/contest/detail?projectId=30055975 |
@talesforce The latest is deployed to prod, but I don't see any change for this one. Can you check? |
I'm not sure, how can I add badges to a test user to see it in the locally deployed app. But as you say it has been deployed to prod, I've checked my profile, and it looks it has not been fixed properly. In fact, I see two cross-controls to close the modal, marked red on the screenshot (one of them is barely visible, as it overlays the modal title): I also think, it might be better to remove those crosses at all, and to make the modal to close on any click in its empty area? |
@ajefts - Not sure if it's a merge issue but I can ask @iversonLv to look at the fix again. BTW, I agree with the suggestion from @birdofpreyru to remove the x altogether and make the modal close on clicking anywhere outside - that's the simplest user experience. |
will take a look |
Expected FixThanks. I've checked both. In the first one I still don't quite like the position of the cross. The second one does not work for me, clicking on the empty space does not close the modal. I'd like to see the following:
P.S.: In case you are not sure how to mock the badges for testing on the locally deployed App, check my last message here: #912 |
Hi, I'd like to fix this issue. But I don't have the permission currently. Could you please give me the permission? |
@shizhouxing The issue is you to work on it. I don't yet have enough permissions myself to invite you to the repo, so just comment in the issue discussion, to pick it up, or say that it is ready for review. |
@birdofpreyru Now it's ready for review. |
* dev: updated display name updated cognitive description test changing react-components version in package.json fix issue 947 Replace URL %20 to dash Fix #905 fixes #940 top menu login/join buttons fixed #904 added check for copilot Update CAB members on the Community Overview page Conflicts: app/settings/email/email.controller.js
The text was updated successfully, but these errors were encountered: