Margins between GUI elements #393


  • New
  • Enhancment
Open
Assigned to nevcairiel
  • Kaliel created this issue Jul 31, 2016

    Actual blocks ("normal", "half", "double") are without margins and their content is does not look good next to each other.

    I made modifications, "AceGUI" when:
    - Width of the content is equal to "width_multiplier" so 170
    - The offset is equal to 16 (varible leftMargin) and is set as the value of the SetPoint
    - usedwidth = framewidth + usedwidth + leftMargin

    I also made a few cosmetic repairs positional value in order to align elements uniform.

    It would be possible to add these changes into the library?

    I attach a modified library AceGUI. All changed / added parts are marked with commentary "-- MSA".

    Thank you
    Kaliel

  • Kaliel added the tags New Enhancment Jul 31, 2016
  • Kaliel added an attachment AceGUI-3.0-modified.patch Jul 31, 2016

    AceGUI-3.0-modified.patch

  • Kaliel added an attachment AceGUI-3.0-modified.zip Jul 31, 2016

    AceGUI-3.0-modified.zip

  • Kaliel added an attachment Jul 31, 2016
    Attachment was deleted Nov 8, 2016
  • Kaliel added an attachment acegui-modified.jpg Jul 31, 2016

    acegui-modified.jpg

    <p>AceGUI - modified</p>

  • Kaliel added an attachment acegui-default.jpg Jul 31, 2016

    acegui-default.jpg

    <p>AceGUI - default</p>

  • nevcairiel posted a comment Jul 31, 2016

    Please provide a patch/diff file so it can be reviewed properly. Full files are not all that helpful.

  • Kaliel posted a comment Jul 31, 2016

    SVN patch uploaded.

  • Kaliel posted a comment Jul 31, 2016

    It would be good add margin as a configuration value. For the entire configuration or for individual nodes.

    Example:

    ...
    args = {
    	general = {
    		name = "Options",
    		type = "group",
    		margin = 16,
    		args = {
    			sec1 = {
    				name = "Info",
    				type = "group",
    				inline = true,
    				...
    
  • nevcairiel posted a comment Jul 31, 2016

    The options table will not get settings like margins, its meant to be generic and unaware of such UI fine-tuning.

    You should also be aware that we really do not like people shipping modified versions of the Ace3 libraries without changing the names of the libraries, because it would affect any and every addon using them when your addon is loaded. So please change that immediately.


    Edited Jul 31, 2016
  • Kaliel posted a comment Jul 31, 2016

    Thank you for reply. It was not necessary immediately to report my project!?
    That's why I asked ... to put it right.


To post a comment, please login or register a new account.