[ https://issues.apache.org/jira/browse/TAP5-2790?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17898740#comment-17898740 ]
Hudson commented on TAP5-2790: ------------------------------ SUCCESS: Integrated in Jenkins build Tapestry » tapestry-javax-java-8-freestyle #59 (See [https://ci-builds.apache.org/job/Tapestry/job/tapestry-javax-java-8-freestyle/59/]) TAP5-2790: Fixing grouped SelectModel for Palette (ben: rev ae8ff3eef5af2a6b02c6c4adbe0c741cb12c65cc) * (edit) tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/Index.java * (edit) tapestry-core/src/test/resources/org/apache/tapestry5/corelib/components/option_groups_precede_ungroup_options.txt * (edit) tapestry-core/src/main/java/org/apache/tapestry5/internal/util/SelectModelRenderer.java * (edit) tapestry-core/src/test/resources/org/apache/tapestry5/corelib/components/option_group_attributes.txt * (edit) tapestry-core/src/test/resources/org/apache/tapestry5/corelib/components/option_groups.txt * (edit) tapestry-core/src/main/coffeescript/META-INF/modules/t5/core/palette.coffee * (edit) tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/PaletteDemo.java * (edit) tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/PaletteTests.java * (add) tapestry-core/src/test/app1/PaletteGroupedDemo.tml * (add) tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/PaletteGroupedDemo.java > Palette with option groups within the SelectModel does not work properly. > ------------------------------------------------------------------------- > > Key: TAP5-2790 > URL: https://issues.apache.org/jira/browse/TAP5-2790 > Project: Tapestry 5 > Issue Type: Bug > Components: tapestry-core > Affects Versions: 5.9.0, 5.8.7 > Reporter: LRein > Assignee: Ben Weidig > Priority: Trivial > Labels: easyfix > Attachments: palette.js > > > Option Groups are displayed correctly in the Palette. If you select options > and then deselect them again, they are not placed in the correct option group. > I am currently correcting the behavior of the Palette when overwriting the > palette.js. But it would be much better if you could adopt the corrected > behavior. > {code:java} > @Contribute(ModuleManager.class) > public static void overrideTapestryCoreModules( > MappedConfiguration<String, Object> configuration, > @Path("META-INF/assets/js/override/palette.js") Resource palette) > { > configuration.add("t5/core/palette", new > JavaScriptModuleConfiguration(palette)); > } {code} > > In Attachments is the change that corrects the behavior of the palette. -- This message was sent by Atlassian Jira (v8.20.10#820010)