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

Incorrect "Additional Boards Manager URLs" dialog field height #881

Closed
per1234 opened this issue Mar 3, 2022 · 2 comments · Fixed by #920
Closed

Incorrect "Additional Boards Manager URLs" dialog field height #881

per1234 opened this issue Mar 3, 2022 · 2 comments · Fixed by #920
Assignees
Labels
conclusion: resolved Issue was resolved criticality: high Of high impact topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project

Comments

@per1234
Copy link
Contributor

per1234 commented Mar 3, 2022

Describe the bug

Arduino IDE supports adding installing arbitrary boards platforms via Boards Manager by adding the URLs of their package index file to the "Additional Boards Manager URLs" preference.

Since users may have several URLs, a dedicated "Additional Boards Manager URLs" dialog is offered with a multi-line field.

🐛 The height of the field is very small, to the point where it is not clear that it is an input field at all.

To Reproduce

  1. Select File > Preferences... from the Arduino IDE menus.
  2. Click the button on the right side of the "Additional Boards Manager URLs" field.

🐛 The height of the field is very small:

image

Expected behavior

Usable input field height in the "Additional Boards Manager URLs" dialog, as it was in previous versions:

image

Desktop

  • OS: Windows 10
  • Version: 2.0.0-rc4-snapshot-0fc7c78
    Date: 2022-03-03T08:39:37.612Z
    CLI Version: 0.21.0 [10107d24]

Additional context

I bisected the bug to 112153f (it does not occur when using the build from the previous commit 69ac1f4).


Originally reported at https://forum.arduino.cc/t/additional-boards-manager-urls-page-cant-read/965207

@per1234 per1234 added topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project labels Mar 3, 2022
@KurtE
Copy link

KurtE commented Mar 3, 2022

Thanks @per1234 was just about to report it after downloading rc4...

I tried resizing the window using the resize part at lower right of text box, but it would only change horizontal size, not vertical.
Windows 10

@ubidefeo ubidefeo added the criticality: high Of high impact label Mar 10, 2022
@VE3EZB
Copy link

VE3EZB commented Mar 11, 2022

I have the same issue:

Processor Intel(R) Core(TM) i5-6500 CPU @ 3.20GHz 3.19 GHz
Installed RAM 16.0 GB (15.9 GB usable)
Device ID F2DD9474-AF46-4403-8077-82B34F6BBDBA
Product ID 00330-80000-00000-AA808
System type 64-bit operating system, x64-based processor

Arduino IDE
Version: 2.0.0-rc5
Date: 2022-03-10T11:09:10.332Z
CLI Version: 0.21.0 [10107d24]

Copyright © 2022 Arduino SA

kittaakos added a commit to kittaakos/arduino-ide that referenced this issue Mar 17, 2022
@per1234 per1234 linked a pull request Mar 18, 2022 that will close this issue
4 tasks
@per1234 per1234 assigned per1234 and unassigned AlbyIanna Mar 18, 2022
kittaakos added a commit to kittaakos/arduino-ide that referenced this issue Mar 27, 2022
kittaakos added a commit to kittaakos/arduino-ide that referenced this issue Mar 28, 2022
fstasi pushed a commit that referenced this issue Apr 4, 2022
* Fixed empty string to URLs conversion

Closes #919.

Signed-off-by: Akos Kitta <[email protected]>

* #881: Fixed height of the 3rd part URLs `textarea`

Closes #881.

Signed-off-by: Akos Kitta <[email protected]>
@per1234 per1234 added the conclusion: resolved Issue was resolved label Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved criticality: high Of high impact topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants