From 97885cde2d46d76631b172be51cf956fabb6fc41 Mon Sep 17 00:00:00 2001 From: Ben Soares Date: Wed, 20 Sep 2023 18:09:45 +0100 Subject: [PATCH] JAL-4265 Use ColorUtils.parseColourString() and adjust documentation to use un-hashed hex-string for consistency with other Jalview uses of parseColourString(). When adding 2 structures and 2 structureimages I am now getting a NPE for String[] files = getStructureFiles(); in AAStructureBindingModel, and when using bgcolour for the first structureimage, losing the residues in the second structure image. Will need to investigate when I'm back --- src/jalview/bin/argparser/Arg.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/jalview/bin/argparser/Arg.java b/src/jalview/bin/argparser/Arg.java index b69ed61..2a3a4a1 100644 --- a/src/jalview/bin/argparser/Arg.java +++ b/src/jalview/bin/argparser/Arg.java @@ -189,7 +189,7 @@ public enum Arg + Arg.COLOUR.argString() + ".", Opt.STRING, Opt.LINKED, Opt.MULTIVALUE, Opt.ALLOWMULTIID), BGCOLOUR(Type.IMAGE, "bgcolor", // being a bit soft on the Americans! - "Applies a background colour to the structure image. Valid values are named colours or #RRGGBB.", + "Applies a background colour to the structure image. Valid values are named colours known to Java or RRGGBB 6 digit hex-string.", Opt.STRING, Opt.LINKED, Opt.MULTIVALUE, Opt.ALLOWMULTIID), /* STRUCTUREIMAGETYPE(Type.IMAGE, -- 1.7.10.2