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

Builder Status: beta test bug reports #80

Closed
ImpulseAdventure opened this issue Nov 13, 2018 · 22 comments
Closed

Builder Status: beta test bug reports #80

ImpulseAdventure opened this issue Nov 13, 2018 · 22 comments
Labels
bug Builder Relating to GUIslice Builder tool

Comments

@ImpulseAdventure
Copy link
Owner

ImpulseAdventure commented Nov 13, 2018

With the launch of the GUIslice Builder (#79) as a beta release, it is likely that issues may be encountered in its operation or associated documentation. This issue will be used to capture user feedback concerning these reports.

This particular entry will also be updated periodically to reflect any known open issues, so as to avoid report duplication.

Bug Report Info

For any reports, please indicate the following details:

  • Operating system (eg. Windows 10)
  • GUIslice version (eg. 0.10.5)
  • GUIslice Builder version (from About menu, eg. 0.10.4-beta5)
  • Target platform; defined in Edit -> Options -> Target Platform (eg. arduino)
  • If a crash occurred, it would be helpful to attach the crash log created in the Builder /logs/ subfolder

Current Known Issues

  • Please refer to the entry associated with the specific release below
@ImpulseAdventure
Copy link
Owner Author

Issue for Builder 0.10.4-beta5 (Windows):

  • Note that the GUIslice.bat batch file had an issue in launching if the JAVA_HOME path included a space (eg. C:\Program Files\...).

FIXED: Builder 0.10.4-beta6 has been uploaded to revise the launch batch script.

Without this update, users might observe the following error upon double-clicking GUIslice.bat:
Windows cannot find 'C:\Program'. Make sure you typed the name correctly, and then try again.

@ImpulseAdventure ImpulseAdventure added the feedback requested Looking for input from users on idea label Dec 21, 2018
@ImpulseAdventure ImpulseAdventure added the Builder Relating to GUIslice Builder tool label Jan 9, 2019
@ImpulseAdventure
Copy link
Owner Author

ImpulseAdventure commented Jan 30, 2019

Summary of Issues in Builder 0.10.4-beta7:

NOTE: Many of these issues have already been resolved in the upcoming Builder 0.11.0-rc1 version, but have been captured here so that users are aware of them in the meantime

  • Internal38: Dragging item in treeview past end of branch (#92)
    • Workaround: avoid dragging past end
    • STATUS: Fixed in 0.11.0-rc1
  • Internal39: Renaming first page can cause generate to fail (#95)
    • Workaround: avoid renaming first page (E_PG_MAIN)
    • STATUS: Fixed in 0.11.0-rc1
  • Internal45: Code gen fails, sometimes duplicating variables for Page$3 and beyond (#95)
    • Workaround: Rename variables as noted in #95
    • STATUS: Fixed in 0.11.0-rc1
  • Internal48: Crash on generate (in macOS) when space in filename
    • Workaround: avoid spaces
    • STATUS: Still open as have not reproduced this yet in development environment (Windows & LINUX).
  • Internal50: Delete not active on new blank page
    • Workaround: Add any element to page first, then delete the page.
    • STATUS: New
  • Internal52: Selecting TextBox element causes crash
    • Workaround: Fix necessary to use this element type
    • STATUS: Fixed in 0.11.0-rc1

@ImpulseAdventure ImpulseAdventure added bug and removed feedback requested Looking for input from users on idea labels Jan 30, 2019
@ImpulseAdventure ImpulseAdventure changed the title GUIslice Builder - beta test bug reports Builder Status: beta test bug reports Feb 12, 2019
@vitormhenrique
Copy link

Hello Calvin and Paul,

I stated using the builder to setup my UI, one thing that it would be really nice is the ability to save colors or palette of colors to reuse them.
Some elements would also beneficial if they had an option to have a "transparent" background.

@vitormhenrique
Copy link

Another cool thing to add would be having the primitives on the tool box (lines, rectangles, circles)
They can be useful to create custom UI's.

@ImpulseAdventure
Copy link
Owner Author

Hi Vitor -- good suggestions!

  • Regarding the palette of recent colors, I understand this has been rolled into the latest Builder release (0.11.0-rc1). Perhaps Paul can send you a binary to test out.
  • For transparent background elements, which elements were you thinking of?
    • In your particular case, would the transparency be useful to reveal an image background or just other elements beneath?
    • GUIslice supports element transparency, but the flag is not currently enabled in most of the Builder elements.
  • For primitives: GUIslice supports two types of drawing APIs, immediate draw mode primitives (eg. DrawFillCircle()) and another set for the creation of GUI elements supporting redraw and touch tracking (eg. ElemCreateBox(), ElemCreateLine(), etc.).
    • At this time, only the Box element is currently supported in the Builder, so we could take a look at adding (at least) the line element as it seems like it would be useful as a divider.
    • In the meantime, I wonder if a "Box" with height or width=1 might be a temporary workaround?

@vitormhenrique
Copy link

Hello Calvin and Paul,

  • I just noticed the recent colors on the last build that Paul gave me, and it is persistent between projects, so that it more than enough I think.

  • My specific scenario was to see elements beneath, for example I was building an ui with two sliders on top of each other, to display my gimbals.

screen shot 2019-02-14 at 11 29 53 am

  • I think lines and circles would be really good, especially if the circles could be Elements that can be updated, ui could use circles to show status of something with different colors, for example on off / read and green (also again on my previous screen shot I would like to add a transparent circle around the sliders...

Like this:

img2

@ImpulseAdventure
Copy link
Owner Author

@vitormhenrique -- from what you describe, I assume you are displaying these virtual gimbals to represent positional status rather than accepting touch control. If so, the most flexible solution is to use the "custom draw callback" approach. In the Builder, create a "Box" element and then enable the custom "Draw Function". This will create a callback routine that enables you to draw the shapes according to your sensor input, while still integrating with the standard GUIslice redraw event hierarchy. Please see ex06_ard_callback for an example of this method and please don't hesitate to let me know if you have any further questions.

@bittenbybytes
Copy link

Hi, I noticed that the builder crashes on code generation once a graph is included in the design, leaving an incomplete source file (missing section markers). It boils down to a simple mistake in typecasting in CodeGenerator.java:1424 replacing TextModel with GraphModel seems to solve the issue.

@Pconti31
Copy link
Contributor

Thanks for the report on XGraph. Fix will be in Builder RC5
Paul--

@DonpK
Copy link

DonpK commented Mar 27, 2019

I had a similar crash problem with the ProgressBar which I reported. I've installed 0.11.0-rc4 and no longer appear to have the same problem. Was the ProgressBar issue addressed in rc4?

@Pconti31
Copy link
Contributor

@DonpK Yes, the progress bar was fixed in rc4.
I actually thought I had fixed XGraph too but that shows what thought gets you...
rc5 should be coming soon.
Paul--

@pdupuy
Copy link

pdupuy commented Jun 25, 2019

OS: Windows 7; GUIslice: 0.12.0 (4769b20); Builder: 0.11.0-rc4; Target: arduino

The image file selection dialog always opens in "Program Files (x86)\GUIsliceBuilder\arduino_res". When there are many images to add and their directory is far removed from the installation directory, adding them is onerous. Possible solutions: add an option to set "Default Image Source Directory"; remember the last directory in which an image was selected.

@pdupuy
Copy link

pdupuy commented Jun 25, 2019

OS: Windows 7; GUIslice: 0.12.0 (4769b20); Builder: 0.11.0-rc4; Target: arduino

Changed "Target's Image Directory" value in General project options is not persisted after saving, closing, and re-opening the project.

@pdupuy
Copy link

pdupuy commented Jun 25, 2019

OS: Windows 7; GUIslice: 0.12.0 (4769b20); Builder: 0.11.0-rc4; Target: arduino

When adding an Image Button, it is required to also choose a file for "selected" or "glow" state ("Choose your Button's Image when pressed"). When not using the glow feature, this seems to add an unnecessary step to the process (again picking the same file). Furthermore, the exported output has to be amended to include a line disabling glow for the image button:

gslc_ElemSetGlowEn(&m_gui,pElemRef,false);

Possible solution: add a project option for default Button Image glow and a "Use Glow" property on Button Image instances; set the "Use Glow" property to the default value on Button Image creation; permit changing it and selecting or removing (or changing) a glow image later.

@pdupuy
Copy link

pdupuy commented Jun 25, 2019

OS: Windows 7; GUIslice: 0.12.0 (4769b20); Builder: 0.11.0-rc4; Target: arduino

When saving or exporting a project file, a confirmation dialog ("Project saved..."; "Code generated...") is generated. This breaks development flow and causes what feels like an unnecessary action; when I save frequently (habitually), the confirmation interruption friction is heightened. Possible solutions: add a project option to "Disable Save confirmation dialog" and "Disable Export confirmation dialog"; remove confirmations of success altogether and only message on failure.

@pdupuy
Copy link

pdupuy commented Jun 25, 2019

OS: Windows 7; GUIslice: 0.12.0 (4769b20); Builder: 0.11.0-rc4; Target: arduino

Deleting a Button Image does not remove the corresponding case in the CbBtnCommon callback function. ImageButton$1 was created and then deleted in the Builder (and it does not appear there); the exported code included a callback for it:

bool CbBtnCommon(void* pvGui,void *pvElemRef,gslc_teTouch eTouch,int16_t nX,int16_t nY)
{
  gslc_tsElemRef* pElemRef = (gslc_tsElemRef*)(pvElemRef);
  gslc_tsElem* pElem = pElemRef->pElem;

  if ( eTouch == GSLC_TOUCH_UP_IN ) {
    // From the element's ID we can determine which button was pressed.
    switch (pElem->nId) {
//<Button Enums !Start!>
      case E_IMGBTN1:
        //TODO- Replace with button handling code
        break;
      case E_IMGBTN2:
        //TODO- Replace with button handling code
        break;

@ImpulseAdventure
Copy link
Owner Author

@pdupuy -- thank you very much for taking the time to file these detailed Builder bug reports! I have integrated them into our internal Builder issue list for review. Note that PaulC (Builder author) is in the midst of a major Builder code-gen rewrite (for 0.13.0) so some enhancements might be pushed to a later update.

@Pconti31
Copy link
Contributor

Pconti31 commented Jul 1, 2019

Changed "Target's Image Directory" value in General project options is not persisted after saving, closing, and re-opening the project.

Assigned bug# 77 and fixed in next release 0.13.0
problem only occured if user failed to press 'enter' key on the field before pressing "'OK" button.

@Pconti31
Copy link
Contributor

Pconti31 commented Jul 1, 2019

The image file selection dialog always opens in "Program Files (x86)\GUIsliceBuilder\arduino_res". >When there are many images to add and their directory is far removed from the installation directory, >adding them is onerous. Possible solutions: add an option to set "Default Image Source Directory"; >remember the last directory in which an image was selected.

Assigned bug#: 76 fixed in next release 0.13.0.
Will now remember last image directory accessed.

@Pconti31
Copy link
Contributor

Pconti31 commented Jul 1, 2019

Deleting a Button Image does not remove the corresponding case in the CbBtnCommon callback function. ImageButton$1 was created and then deleted in the Builder (and it does not appear there); the exported code included a callback for it:

Assigned bug#80: fixed in next release of builder 0.13.0

@Pconti31
Copy link
Contributor

Pconti31 commented Jul 1, 2019

When saving or exporting a project file, a confirmation dialog ("Project saved..."; "Code generated...") is >generated. This breaks development flow and causes what feels like an unnecessary action; when I save >frequently (habitually), the confirmation interruption friction is heightened. Possible solutions: add a >project option to "Disable Save confirmation dialog" and "Disable Export confirmation dialog"; remove >confirmations of success altogether and only message on failure.

Assigned bug#80: Fixed in next release of builder
implemented a status bar instead of confirmation dialogs.

@ImpulseAdventure
Copy link
Owner Author

Closing beta test thread given that Builder 0.13.0 was released.

We will keep track of issues with the Builder in its own companion repository. Please file any problem reports at the Builder repository noted below:
GUIslice Builder - Issues

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Builder Relating to GUIslice Builder tool
Projects
None yet
Development

No branches or pull requests

6 participants