Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Me: Security - 2FA - Compact notice close link is not aligned #21

Closed
hoverduck opened this issue Nov 17, 2015 · 1 comment
Closed

Me: Security - 2FA - Compact notice close link is not aligned #21

hoverduck opened this issue Nov 17, 2015 · 1 comment
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Good First Issue Small, contained issues. Security [Type] Bug

Comments

@hoverduck
Copy link
Contributor

I was running through the steps to disable and then re-enable 2FA today and came to the step where it asked me to type my verification code. Since I had just done so to enter /me, I had to wait a bit. I got this notice and the X was underlined and not aligned to the top right. It was also is-info and would probably make more sense as a is-warning.

image

Error message too:
image

Should be simple CSS and HTML class fixes here

@hoverduck hoverduck added [Type] Bug [Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Design Security labels Nov 17, 2015
@lancewillett lancewillett added the Good First Issue Small, contained issues. label Dec 4, 2015
@ebinnion
Copy link
Contributor

ebinnion commented Dec 7, 2015

This seems to have already been fixed, likely by pre-OSS PR 12107.

@ebinnion ebinnion closed this as completed Dec 7, 2015
sirreal pushed a commit that referenced this issue Dec 5, 2018
jsx-classname-namespace: Mark element in render as root only if returned
sirreal pushed a commit that referenced this issue Dec 5, 2018
jsx-classname-namespace: Mark element in render as root only if returned
sirreal pushed a commit that referenced this issue Dec 6, 2018
jsx-classname-namespace: Mark element in render as root only if returned
sirreal pushed a commit that referenced this issue Dec 11, 2018
jsx-classname-namespace: Mark element in render as root only if returned
sgomes pushed a commit that referenced this issue Jan 29, 2020
* First pass at allowing reloading

* Build after installing, so we can use GitHub branches as dependencies

* Use npm instead of make for postinstall

* Try using postinstall-build for postinstall?

* Package: need to include index.js, so it can be built.

* Package: need to include Makefile, too.

* Wait for a message from the iframe to trigger onload()

* Run the test suite twice - once normally, then once after reload.

* Add myself as a contributor

* Fix some code formatting bugs

* Let prepublish build the lib when installed as a dependency

* Bump package version to v4.0.0

* Don't add index.js and Makefile to npm package
sgomes pushed a commit that referenced this issue Feb 18, 2020
* First pass at allowing reloading

* Build after installing, so we can use GitHub branches as dependencies

* Use npm instead of make for postinstall

* Try using postinstall-build for postinstall?

* Package: need to include index.js, so it can be built.

* Package: need to include Makefile, too.

* Wait for a message from the iframe to trigger onload()

* Run the test suite twice - once normally, then once after reload.

* Add myself as a contributor

* Fix some code formatting bugs

* Let prepublish build the lib when installed as a dependency

* Bump package version to v4.0.0

* Don't add index.js and Makefile to npm package
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] User & Account Settings (/me) Settings and tools for managing your WordPress.com user account. Good First Issue Small, contained issues. Security [Type] Bug
Projects
None yet
Development

No branches or pull requests

4 participants