Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

Make 'ignoreFocusOut' true for all input #1372

Closed
2 tasks
Jakeeyturner opened this issue Sep 4, 2019 · 1 comment · Fixed by #1408
Closed
2 tasks

Make 'ignoreFocusOut' true for all input #1372

Jakeeyturner opened this issue Sep 4, 2019 · 1 comment · Fixed by #1408
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@Jakeeyturner
Copy link
Contributor

Description

We should probably use ignoreFocusOut: true for all our inputs. That way, a user has to explicitly escape the input to make it go away.

MGK hit the problem when taking screenshots, that some of the inputs disappeared when he pressed the 'print screen' key.

I'm not sure of any reason why we would have ignoreFocusOut: false, or why the option varies for different inputs.

Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Existing issues

Log File

Your Environment

  • IBM Blockchain Platform Extension Version:
  • Visual Studio Code version:
  • Environment name and version (e.g. Node.js v8.12.0, npm v6.4.1):
  • Operating System and version:
  • Link to your project:
@Jakeeyturner Jakeeyturner added the enhancement New feature or request label Sep 4, 2019
@cazfletch cazfletch added this to the Sprint 16 milestone Sep 11, 2019
@Jakeeyturner Jakeeyturner added the good first issue Good for newcomers label Sep 12, 2019
@erin-hughes
Copy link
Contributor

working on this now 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants