[LyX/master] Add dark theme colors to semantic colors (#8325)
Juergen Spitzmueller
spitz at lyx.org
Sat Dec 12 09:26:55 UTC 2020
commit 704331fdcd18129e74ff9ac089c044571e707eb7
Author: Juergen Spitzmueller <spitz at lyx.org>
Date: Sat Dec 12 10:56:05 2020 +0100
Add dark theme colors to semantic colors (#8325)
---
lib/scripts/prefs2prefs_prefs.py | 7 +-
src/Color.cpp | 245 +++++++++++++++++++----------------
src/ColorSet.h | 19 ++-
src/LyXAction.cpp | 5 +-
src/LyXRC.cpp | 20 ++-
src/frontends/qt/ColorCache.cpp | 14 ++-
src/frontends/qt/ColorCache.h | 3 +
src/frontends/qt/GuiApplication.cpp | 11 +-
src/frontends/qt/GuiPrefs.cpp | 3 +-
9 files changed, 194 insertions(+), 133 deletions(-)
diff --git a/lib/scripts/prefs2prefs_prefs.py b/lib/scripts/prefs2prefs_prefs.py
index fcbf6fb..88b0041 100644
--- a/lib/scripts/prefs2prefs_prefs.py
+++ b/lib/scripts/prefs2prefs_prefs.py
@@ -137,6 +137,10 @@
# Incremented to format 34, by yuriy
# Rename *.kmap files for Cyrillic languages
+# Incremented to format 35, by spitz
+# \set_color now takes three arguments
+# \set_color lyxname x11hexname x11darkhexname
+
# NOTE: The format should also be updated in LYXRC.cpp and
# in configure.py.
@@ -499,5 +503,6 @@ conversions = [
[ 31, []],
[ 32, []],
[ 33, []],
- [ 34, [rename_cyrillic_kmap_files]]
+ [ 34, [rename_cyrillic_kmap_files]],
+ [ 35, []]
]
diff --git a/src/Color.cpp b/src/Color.cpp
index e68b8b4..6f41350 100644
--- a/src/Color.cpp
+++ b/src/Color.cpp
@@ -42,6 +42,7 @@ struct ColorSet::ColorEntry {
char const * guiname;
char const * latexname;
char const * x11hexname;
+ char const * x11darkhexname;
char const * lyxname;
};
@@ -225,115 +226,115 @@ ColorSet::ColorSet()
char const * RoyalBlue = "#4169e1";
//char const * grey90 = "#e5e5e5";
- // ColorCode, gui, latex, x11hexname, lyx
+ // ColorCode, gui, latex, x11hexname, x11darkhexname, lyx
// Warning: several of these entries are overridden in GuiApplication constructor
static ColorEntry const items[] = {
- { Color_none, N_("none"), "none", black, "none" },
- { Color_black, N_("black"), "black", black, "black" },
- { Color_white, N_("white"), "white", white, "white" },
- { Color_blue, N_("blue"), "blue", blue, "blue" },
- { Color_brown, N_("brown"), "brown", brown, "brown" },
- { Color_cyan, N_("cyan"), "cyan", cyan, "cyan" },
- { Color_darkgray, N_("darkgray"), "darkgray", darkgray, "darkgray" },
- { Color_gray, N_("gray"), "gray", gray, "gray" },
- { Color_green, N_("green"), "green", green, "green" },
- { Color_lightgray, N_("lightgray"), "lightgray", lightgray, "lightgray" },
- { Color_lime, N_("lime"), "lime", lime, "lime" },
- { Color_magenta, N_("magenta"), "magenta", magenta, "magenta" },
- { Color_olive, N_("olive"), "olive", olive, "olive" },
- { Color_orange, N_("orange"), "orange", orange, "orange" },
- { Color_pink, N_("pink"), "pink", pink, "pink" },
- { Color_purple, N_("purple"), "purple", purple, "purple" },
- { Color_red, N_("red"), "red", red, "red" },
- { Color_teal, N_("teal"), "teal", teal, "teal" },
- { Color_violet, N_("violet"), "violet", violet, "violet" },
- { Color_yellow, N_("yellow"), "yellow", yellow, "yellow" },
- { Color_cursor, N_("cursor"), "cursor", black, "cursor" },
- { Color_background, N_("background"), "background", Linen, "background" },
- { Color_foreground, N_("text"), "foreground", black, "foreground" },
- { Color_selection, N_("selection"), "selection", "#add8e6", "selection" },
- { Color_selectiontext, N_("selected text"), "selectiontext", black, "selectiontext" },
- { Color_latex, N_("LaTeX text"), "latex", DarkRed, "latex" },
- { Color_textlabel1, N_("Text label 1"), "textlabel1", blue, "textlabel1" },
- { Color_textlabel2, N_("Text label 2"), "textlabel2", Green, "textlabel2" },
- { Color_textlabel3, N_("Text label 3"), "textlabel3", magenta, "textlabel3" },
+ { Color_none, N_("none"), "none", black, black, "none" },
+ { Color_black, N_("black"), "black", black, black, "black" },
+ { Color_white, N_("white"), "white", white, white, "white" },
+ { Color_blue, N_("blue"), "blue", blue, blue, "blue" },
+ { Color_brown, N_("brown"), "brown", brown, brown, "brown" },
+ { Color_cyan, N_("cyan"), "cyan", cyan, cyan, "cyan" },
+ { Color_darkgray, N_("darkgray"), "darkgray", darkgray, darkgray, "darkgray" },
+ { Color_gray, N_("gray"), "gray", gray, gray, "gray" },
+ { Color_green, N_("green"), "green", green, green, "green" },
+ { Color_lightgray, N_("lightgray"), "lightgray", lightgray, lightgray, "lightgray" },
+ { Color_lime, N_("lime"), "lime", lime, lime, "lime" },
+ { Color_magenta, N_("magenta"), "magenta", magenta, magenta, "magenta" },
+ { Color_olive, N_("olive"), "olive", olive, olive, "olive" },
+ { Color_orange, N_("orange"), "orange", orange, orange, "orange" },
+ { Color_pink, N_("pink"), "pink", pink, pink, "pink" },
+ { Color_purple, N_("purple"), "purple", purple, purple, "purple" },
+ { Color_red, N_("red"), "red", red, red, "red" },
+ { Color_teal, N_("teal"), "teal", teal, teal, "teal" },
+ { Color_violet, N_("violet"), "violet", violet, violet, "violet" },
+ { Color_yellow, N_("yellow"), "yellow", yellow, yellow, "yellow" },
+ { Color_cursor, N_("cursor"), "cursor", black, Linen, "cursor" },
+ { Color_background, N_("background"), "background", Linen, black, "background" },
+ { Color_foreground, N_("text"), "foreground", black, Linen, "foreground" },
+ { Color_selection, N_("selection"), "selection", "#add8e6", "#add8e6", "selection" },
+ { Color_selectiontext, N_("selected text"), "selectiontext", black, black, "selectiontext" },
+ { Color_latex, N_("LaTeX text"), "latex", DarkRed, "#f2af7d", "latex" },
+ { Color_textlabel1, N_("Text label 1"), "textlabel1", blue, "#86a4ff", "textlabel1" },
+ { Color_textlabel2, N_("Text label 2"), "textlabel2", Green, green, "textlabel2" },
+ { Color_textlabel3, N_("Text label 3"), "textlabel3", magenta, magenta, "textlabel3" },
{ Color_inlinecompletion, N_("inline completion"),
- "inlinecompletion", grey60, "inlinecompletion" },
+ "inlinecompletion", grey60, grey40, "inlinecompletion" },
{ Color_nonunique_inlinecompletion, N_("non-unique inline completion"),
- "nonuniqueinlinecompletion", grey80, "nonuniqueinlinecompletion" },
- { Color_preview, N_("previewed snippet"), "preview", black, "preview" },
- { Color_notelabel, N_("note label"), "note", yellow, "note" },
- { Color_notebg, N_("note background"), "notebg", yellow, "notebg" },
- { Color_commentlabel, N_("comment label"), "comment", magenta, "comment" },
- { Color_commentbg, N_("comment background"), "commentbg", Linen, "commentbg" },
- { Color_greyedoutlabel, N_("greyedout inset label"), "greyedout", "#ff0080", "greyedout" },
- { Color_greyedouttext, N_("greyedout inset text"), "greyedouttext", grey80, "greyedouttext" },
- { Color_greyedoutbg, N_("greyedout inset background"), "greyedoutbg", Linen, "greyedoutbg" },
- { Color_phantomtext, N_("phantom inset text"), "phantomtext", "#7f7f7f", "phantomtext" },
- { Color_shadedbg, N_("shaded box"), "shaded", "#ff0000", "shaded" },
- { Color_listingsbg, N_("listings background"), "listingsbg", white, "listingsbg" },
- { Color_branchlabel, N_("branch label"), "branchlabel", "#c88000", "branchlabel" },
- { Color_footlabel, N_("footnote label"), "footlabel", "#00aaff", "footlabel" },
- { Color_indexlabel, N_("index label"), "indexlabel", Green, "indexlabel" },
- { Color_marginlabel, N_("margin note label"), "marginlabel", "#aa55ff", "marginlabel" },
- { Color_urllabel, N_("URL label"), "urllabel", blue, "urllabel" },
- { Color_urltext, N_("URL text"), "urltext", blue, "urltext" },
- { Color_depthbar, N_("depth bar"), "depthbar", IndianRed, "depthbar" },
- { Color_scroll, N_("scroll indicator"), "scroll", IndianRed, "scroll" },
- { Color_language, N_("language"), "language", blue, "language" },
- { Color_command, N_("command inset"), "command", black, "command" },
- { Color_commandbg, N_("command inset background"), "commandbg", "#f0ffff", "commandbg" },
- { Color_commandframe, N_("command inset frame"), "commandframe", black, "commandframe" },
- { Color_command_broken, N_("command inset (broken reference)"), "command", white, "command_broken" },
- { Color_buttonbg_broken, N_("button background (broken reference)"), "commandbg", red, "commandbg_broken" },
- { Color_buttonframe_broken, N_("button frame (broken reference)"), "commandframe", red, "commandframe_broken" },
- { Color_buttonhoverbg_broken, N_("button background (broken reference) under focus"), "buttonhoverbg", "#DB0B0B", "buttonhoverbg_broken" },
- { Color_special, N_("special character"), "special", RoyalBlue, "special" },
- { Color_math, N_("math"), "math", "#00008B", "math" },
- { Color_mathbg, N_("math background"), "mathbg", Linen, "mathbg" },
- { Color_graphicsbg, N_("graphics background"), "graphicsbg", Linen, "graphicsbg" },
- { Color_mathmacrobg, N_("math macro background"), "mathmacrobg", Linen, "mathmacrobg" },
- { Color_mathframe, N_("math frame"), "mathframe", magenta, "mathframe" },
- { Color_mathcorners, N_("math corners"), "mathcorners", Linen, "mathcorners" },
- { Color_mathline, N_("math line"), "mathline", blue, "mathline" },
- { Color_mathmacrobg, N_("math macro background"), "mathmacrobg", "#ede2d8", "mathmacrobg" },
- { Color_mathmacrohoverbg, N_("math macro hovered background"), "mathmacrohoverbg", "#cdc3b8", "mathmacrohoverbg" },
- { Color_mathmacrolabel, N_("math macro label"), "mathmacrolabel", "#a19992", "mathmacrolabel" },
- { Color_mathmacroframe, N_("math macro frame"), "mathmacroframe", "#ede2d8", "mathmacroframe" },
- { Color_mathmacroblend, N_("math macro blended out"), "mathmacroblend", black, "mathmacroblend" },
- { Color_mathmacrooldarg, N_("math macro old parameter"), "mathmacrooldarg", grey80, "mathmacrooldarg" },
- { Color_mathmacronewarg, N_("math macro new parameter"), "mathmacronewarg", black, "mathmacronewarg" },
- { Color_collapsible, N_("collapsible inset text"), "collapsible", DarkRed, "collapsible" },
- { Color_collapsibleframe, N_("collapsible inset frame"), "collapsibleframe", IndianRed, "collapsibleframe" },
- { Color_insetbg, N_("inset background"), "insetbg", grey80, "insetbg" },
- { Color_insetframe, N_("inset frame"), "insetframe", IndianRed, "insetframe" },
- { Color_error, N_("LaTeX error"), "error", red, "error" },
- { Color_eolmarker, N_("end-of-line marker"), "eolmarker", Brown, "eolmarker" },
- { Color_appendix, N_("appendix marker"), "appendix", Brown, "appendix" },
- { Color_changebar, N_("change bar"), "changebar", blue, "changebar" },
- { Color_deletedtext, N_("deleted text (output)"), "deletedtext", "#ff0000", "deletedtext" },
- { Color_addedtext, N_("added text (output)"), "addedtext", "#0000ff", "addedtext" },
- { Color_addedtextauthor1, N_("added text (workarea, 1st author)"), "changedtextauthor1", "#0000ff", "changedtextauthor1" },
- { Color_addedtextauthor2, N_("added text (workarea, 2nd author)"), "changedtextauthor2", "#ff00ff", "changedtextauthor2" },
- { Color_addedtextauthor3, N_("added text (workarea, 3rd author)"), "changedtextauthor3", "#ff0000", "changedtextauthor3" },
- { Color_addedtextauthor4, N_("added text (workarea, 4th author)"), "changedtextauthor4", "#aa00ff", "changedtextauthor4" },
- { Color_addedtextauthor5, N_("added text (workarea, 5th author)"), "changedtextauthor5", "#55aa00", "changedtextauthor5" },
- { Color_deletedtextmodifier, N_("deleted text modifier (workarea)"), "deletedtextmodifier", white, "deletedtextmodifier" },
- { Color_added_space, N_("added space markers"), "added_space", Brown, "added_space" },
- { Color_tabularline, N_("table line"), "tabularline", black, "tabularline" },
- { Color_tabularonoffline, N_("table on/off line"), "tabularonoffline", "#b0c4de", "tabularonoffline" },
- { Color_bottomarea, N_("bottom area"), "bottomarea", grey40, "bottomarea" },
- { Color_newpage, N_("new page"), "newpage", blue, "newpage" },
- { Color_pagebreak, N_("page break / line break"), "pagebreak", RoyalBlue, "pagebreak" },
- { Color_buttonframe, N_("button frame"), "buttonframe", "#dcd2c8", "buttonframe" },
- { Color_buttonbg, N_("button background"), "buttonbg", "#dcd2c8", "buttonbg" },
- { Color_buttonhoverbg, N_("button background under focus"), "buttonhoverbg", "#C7C7CA", "buttonhoverbg" },
- { Color_paragraphmarker, N_("paragraph marker"), "paragraphmarker", grey80, "paragraphmarker"},
- { Color_previewframe, N_("preview frame"), "previewframe", black, "previewframe"},
- { Color_inherit, N_("inherit"), "inherit", black, "inherit" },
- { Color_regexpframe, N_("regexp frame"), "regexpframe", Green, "regexpframe" },
- { Color_ignore, N_("ignore"), "ignore", black, "ignore" },
- { Color_ignore, nullptr, nullptr, nullptr, nullptr }
+ "nonuniqueinlinecompletion", grey80, grey60, "nonuniqueinlinecompletion" },
+ { Color_preview, N_("previewed snippet"), "preview", black, Linen, "preview" },
+ { Color_notelabel, N_("note label"), "note", yellow, "#FF6200", "note" },
+ { Color_notebg, N_("note background"), "notebg", yellow, "#5b5903", "notebg" },
+ { Color_commentlabel, N_("comment label"), "comment", magenta, olive, "comment" },
+ { Color_commentbg, N_("comment background"), "commentbg", Linen, black, "commentbg" },
+ { Color_greyedoutlabel, N_("greyedout inset label"), "greyedout", "#ff0080", "#ff0080", "greyedout" },
+ { Color_greyedouttext, N_("greyedout inset text"), "greyedouttext", grey80, grey40, "greyedouttext" },
+ { Color_greyedoutbg, N_("greyedout inset background"), "greyedoutbg", Linen, black, "greyedoutbg" },
+ { Color_phantomtext, N_("phantom inset text"), "phantomtext", "#7f7f7f", "#7f7f7f", "phantomtext" },
+ { Color_shadedbg, N_("shaded box"), "shaded", "#ff0000", "#f2af7d", "shaded" },
+ { Color_listingsbg, N_("listings background"), "listingsbg", white, black, "listingsbg" },
+ { Color_branchlabel, N_("branch label"), "branchlabel", "#c88000", "#c88000", "branchlabel" },
+ { Color_footlabel, N_("footnote label"), "footlabel", "#00aaff", blue, "footlabel" },
+ { Color_indexlabel, N_("index label"), "indexlabel", Green, teal, "indexlabel" },
+ { Color_marginlabel, N_("margin note label"), "marginlabel", "#aa55ff", violet, "marginlabel" },
+ { Color_urllabel, N_("URL label"), "urllabel", blue, blue, "urllabel" },
+ { Color_urltext, N_("URL text"), "urltext", blue, "#86a4ff", "urltext" },
+ { Color_depthbar, N_("depth bar"), "depthbar", IndianRed, IndianRed, "depthbar" },
+ { Color_scroll, N_("scroll indicator"), "scroll", IndianRed, IndianRed, "scroll" },
+ { Color_language, N_("language"), "language", blue, "#86a4ff", "language" },
+ { Color_command, N_("command inset"), "command", black, black, "command" },
+ { Color_commandbg, N_("command inset background"), "commandbg", "#f0ffff", "#f0ffff", "commandbg" },
+ { Color_commandframe, N_("command inset frame"), "commandframe", black, Linen, "commandframe" },
+ { Color_command_broken, N_("command inset (broken reference)"), "command", white, white, "command_broken" },
+ { Color_buttonbg_broken, N_("button background (broken reference)"), "commandbg", red, red, "commandbg_broken" },
+ { Color_buttonframe_broken, N_("button frame (broken reference)"), "commandframe", red, red, "commandframe_broken" },
+ { Color_buttonhoverbg_broken, N_("button background (broken reference) under focus"), "buttonhoverbg", "#DB0B0B", "#DB0B0B", "buttonhoverbg_broken" },
+ { Color_special, N_("special character"), "special", RoyalBlue, RoyalBlue, "special" },
+ { Color_math, N_("math"), "math", "#00008B", "#85F0FE", "math" },
+ { Color_mathbg, N_("math background"), "mathbg", Linen, black, "mathbg" },
+ { Color_graphicsbg, N_("graphics background"), "graphicsbg", Linen, black, "graphicsbg" },
+ { Color_mathmacrobg, N_("math macro background"), "mathmacrobg", Linen, black, "mathmacrobg" },
+ { Color_mathframe, N_("math frame"), "mathframe", magenta, magenta, "mathframe" },
+ { Color_mathcorners, N_("math corners"), "mathcorners", Linen, black, "mathcorners" },
+ { Color_mathline, N_("math line"), "mathline", blue, "#86a4ff", "mathline" },
+ { Color_mathmacrobg, N_("math macro background"), "mathmacrobg", "#ede2d8", black, "mathmacrobg" },
+ { Color_mathmacrohoverbg, N_("math macro hovered background"), "mathmacrohoverbg", "#cdc3b8", grey80, "mathmacrohoverbg" },
+ { Color_mathmacrolabel, N_("math macro label"), "mathmacrolabel", "#a19992", "#a19992", "mathmacrolabel" },
+ { Color_mathmacroframe, N_("math macro frame"), "mathmacroframe", "#ede2d8", black, "mathmacroframe" },
+ { Color_mathmacroblend, N_("math macro blended out"), "mathmacroblend", black, Linen, "mathmacroblend" },
+ { Color_mathmacrooldarg, N_("math macro old parameter"), "mathmacrooldarg", grey80, grey40, "mathmacrooldarg" },
+ { Color_mathmacronewarg, N_("math macro new parameter"), "mathmacronewarg", black, Linen, "mathmacronewarg" },
+ { Color_collapsible, N_("collapsible inset text"), "collapsible", DarkRed, DarkRed, "collapsible" },
+ { Color_collapsibleframe, N_("collapsible inset frame"), "collapsibleframe", IndianRed, IndianRed, "collapsibleframe" },
+ { Color_insetbg, N_("inset background"), "insetbg", grey80, grey80, "insetbg" },
+ { Color_insetframe, N_("inset frame"), "insetframe", IndianRed, IndianRed, "insetframe" },
+ { Color_error, N_("LaTeX error"), "error", red, DarkRed, "error" },
+ { Color_eolmarker, N_("end-of-line marker"), "eolmarker", Brown, Brown, "eolmarker" },
+ { Color_appendix, N_("appendix marker"), "appendix", Brown, Brown, "appendix" },
+ { Color_changebar, N_("change bar"), "changebar", blue, "#86a4ff", "changebar" },
+ { Color_deletedtext, N_("deleted text (output)"), "deletedtext", "#ff0000", "#ff0000", "deletedtext" },
+ { Color_addedtext, N_("added text (output)"), "addedtext", "#0000ff", "#0000ff", "addedtext" },
+ { Color_addedtextauthor1, N_("added text (workarea, 1st author)"), "changedtextauthor1", "#0000ff", "#86a4ff", "changedtextauthor1" },
+ { Color_addedtextauthor2, N_("added text (workarea, 2nd author)"), "changedtextauthor2", "#ff00ff", "#ee86ee", "changedtextauthor2" },
+ { Color_addedtextauthor3, N_("added text (workarea, 3rd author)"), "changedtextauthor3", "#ff0000", "#ea8989", "changedtextauthor3" },
+ { Color_addedtextauthor4, N_("added text (workarea, 4th author)"), "changedtextauthor4", "#aa00ff", "#c371ec", "changedtextauthor4" },
+ { Color_addedtextauthor5, N_("added text (workarea, 5th author)"), "changedtextauthor5", "#55aa00", "#acd780", "changedtextauthor5" },
+ { Color_deletedtextmodifier, N_("deleted text modifier (workarea)"), "deletedtextmodifier", white, white, "deletedtextmodifier" },
+ { Color_added_space, N_("added space markers"), "added_space", Brown, Brown, "added_space" },
+ { Color_tabularline, N_("table line"), "tabularline", black, Linen, "tabularline" },
+ { Color_tabularonoffline, N_("table on/off line"), "tabularonoffline", "#b0c4de", "#23497b", "tabularonoffline" },
+ { Color_bottomarea, N_("bottom area"), "bottomarea", grey40, grey80, "bottomarea" },
+ { Color_newpage, N_("new page"), "newpage", blue, "#86a4ff", "newpage" },
+ { Color_pagebreak, N_("page break / line break"), "pagebreak", RoyalBlue, RoyalBlue, "pagebreak" },
+ { Color_buttonframe, N_("button frame"), "buttonframe", "#dcd2c8", "#dcd2c8", "buttonframe" },
+ { Color_buttonbg, N_("button background"), "buttonbg", "#dcd2c8", "#dcd2c8", "buttonbg" },
+ { Color_buttonhoverbg, N_("button background under focus"), "buttonhoverbg", "#C7C7CA", "#C7C7CA", "buttonhoverbg" },
+ { Color_paragraphmarker, N_("paragraph marker"), "paragraphmarker", grey80, grey40, "paragraphmarker"},
+ { Color_previewframe, N_("preview frame"), "previewframe", black, Linen, "previewframe"},
+ { Color_inherit, N_("inherit"), "inherit", black, Linen, "inherit" },
+ { Color_regexpframe, N_("regexp frame"), "regexpframe", Green, green, "regexpframe" },
+ { Color_ignore, N_("ignore"), "ignore", black, Linen, "ignore" },
+ { Color_ignore, nullptr, nullptr, nullptr, nullptr, nullptr }
};
for (int i = 0; items[i].guiname; ++i)
@@ -348,6 +349,7 @@ void ColorSet::fill(ColorEntry const & entry)
in.lyxname = entry.lyxname;
in.latexname = entry.latexname;
in.x11hexname = entry.x11hexname;
+ in.x11darkhexname = entry.x11darkhexname;
in.guiname = entry.guiname;
infotab[entry.lcolor] = in;
lyxcolors[entry.lyxname] = entry.lcolor;
@@ -364,16 +366,29 @@ docstring const ColorSet::getGUIName(ColorCode c) const
}
-string const ColorSet::getX11HexName(ColorCode c) const
+string const ColorSet::getX11HexName(ColorCode c, bool const darkmode) const
{
InfoTab::const_iterator it = infotab.find(c);
if (it != infotab.end())
- return it->second.x11hexname;
+ return darkmode ? it->second.x11darkhexname : it->second.x11hexname;
lyxerr << "LyX internal error: Missing color"
" entry in Color.cpp for " << c << '\n'
<< "Using black." << endl;
- return "black";
+ return darkmode ? "#faf0e6" : "black";
+}
+
+
+pair<string, string> const ColorSet::getAllX11HexNames(ColorCode c) const
+{
+ InfoTab::const_iterator it = infotab.find(c);
+ if (it != infotab.end())
+ return make_pair(it->second.x11hexname, it->second.x11darkhexname);
+
+ lyxerr << "LyX internal error: Missing color"
+ " entry in Color.cpp for " << c << '\n'
+ << "Using black." << endl;
+ return make_pair("black", "#faf0e6");
}
@@ -395,7 +410,8 @@ string const ColorSet::getLyXName(ColorCode c) const
}
-bool ColorSet::setColor(ColorCode col, string const & x11hexname)
+bool ColorSet::setColor(ColorCode col, string const & x11hexname,
+ string const & x11darkhexname)
{
InfoTab::iterator it = infotab.find(col);
if (it == infotab.end()) {
@@ -410,12 +426,15 @@ bool ColorSet::setColor(ColorCode col, string const & x11hexname)
return false;
}
- it->second.x11hexname = x11hexname;
+ if (!x11hexname.empty())
+ it->second.x11hexname = x11hexname;
+ it->second.x11darkhexname = (x11darkhexname.empty()) ? x11hexname : x11darkhexname;
return true;
}
-bool ColorSet::setColor(string const & lyxname, string const & x11hexname)
+bool ColorSet::setColor(string const & lyxname, string const & x11hexname,
+ string const & x11darkhexname)
{
string const lcname = ascii_lowercase(lyxname);
if (lyxcolors.find(lcname) == lyxcolors.end()) {
@@ -424,13 +443,13 @@ bool ColorSet::setColor(string const & lyxname, string const & x11hexname)
addColor(static_cast<ColorCode>(infotab.size()), lcname);
}
- return setColor(lyxcolors[lcname], x11hexname);
+ return setColor(lyxcolors[lcname], x11hexname, x11darkhexname);
}
void ColorSet::addColor(ColorCode c, string const & lyxname)
{
- ColorEntry ce = { c, "", "", "", lyxname.c_str() };
+ ColorEntry ce = { c, "", "", "", "", lyxname.c_str() };
fill(ce);
}
diff --git a/src/ColorSet.h b/src/ColorSet.h
index 31e9312..d688f44 100644
--- a/src/ColorSet.h
+++ b/src/ColorSet.h
@@ -49,21 +49,28 @@ public:
ColorSet();
/** set the given LyX color to the color defined by the X11 hex name given
- * \returns true if successful.
+ * \returns true if successful. The optional third argument passes
+ * a color for dark mode.
*/
- bool setColor(ColorCode col, std::string const & x11hexname);
+ bool setColor(ColorCode col, std::string const & x11hexname,
+ std::string const & x11darkhexname = std::string());
/** set the given LyX color to the color defined by the X11
* hex name given \returns true if successful. A new color entry
- * is created if the color is unknown
+ * is created if the color is unknown. The optional third argument passes
+ * a color for dark mode.
*/
- bool setColor(std::string const & lyxname, std::string const & x11hexname);
+ bool setColor(std::string const & lyxname, std::string const & x11hexname,
+ std::string const & x11darkhexname = std::string());
/// Get the GUI name of \c color.
docstring const getGUIName(ColorCode c) const;
/// Get the X11 hexname of \c color.
- std::string const getX11HexName(ColorCode c) const;
+ std::string const getX11HexName(ColorCode c, bool const darkmode = false) const;
+
+ /// Get the X11 hexname of \c color.
+ std::pair<std::string, std::string> const getAllX11HexNames(ColorCode c) const;
/// Get the LaTeX name of \c color.
std::string const getLaTeXName(ColorCode c) const;
@@ -88,6 +95,8 @@ private:
std::string latexname;
/// the name for X11
std::string x11hexname;
+ /// matching X11 color for dark mode
+ std::string x11darkhexname;
/// the name for LyX
std::string lyxname;
};
diff --git a/src/LyXAction.cpp b/src/LyXAction.cpp
index 7c89000..25bf049 100644
--- a/src/LyXAction.cpp
+++ b/src/LyXAction.cpp
@@ -3680,10 +3680,11 @@ void LyXAction::init()
/*!
* \var lyx::FuncCode lyx::LFUN_SET_COLOR
- * \li Action: Set the given LyX color to the color defined by the X11 name given.
+ * \li Action: Set the given LyX color to the color defined by the X11 name given,
+ * and optionally a specific color for dark mode.
* \li Notion: A new color entry is created if the color is unknown.
Color names can be stored as a part of user settings.
- * \li Syntax: set-color <LYX_NAME> <X11_NAME>
+ * \li Syntax: set-color <LYX_NAME> <X11_NAME> [<X11_DARKNAME>]
* \li Origin: SLior, 11 Jun 2000
* \endvar
*/
diff --git a/src/LyXRC.cpp b/src/LyXRC.cpp
index 4fd5a28..52a7168 100644
--- a/src/LyXRC.cpp
+++ b/src/LyXRC.cpp
@@ -60,7 +60,7 @@ namespace {
// The format should also be updated in configure.py, and conversion code
// should be added to prefs2prefs_prefs.py.
-static unsigned int const LYXRC_FILEFORMAT = 34; // yuriy: rename kmap files
+static unsigned int const LYXRC_FILEFORMAT = 35; // spitz: dark mode colors
// when adding something to this array keep it sorted!
LexerKeyword lyxrcTags[] = {
{ "\\accept_compound", LyXRC::RC_ACCEPT_COMPOUND },
@@ -657,6 +657,10 @@ LyXRC::ReturnValues LyXRC::read(Lexer & lexrc, bool check_format)
}
string const x11_name = lexrc.getString();
+ string x11_darkname = x11_name;
+ if (lexrc.next())
+ x11_darkname = lexrc.getString();
+
ColorCode const col =
lcolor.getFromLyXName(lyx_name);
if (col == Color_none ||
@@ -664,9 +668,10 @@ LyXRC::ReturnValues LyXRC::read(Lexer & lexrc, bool check_format)
col == Color_ignore)
break;
- if (!lcolor.setColor(col, x11_name))
+ if (!lcolor.setColor(col, x11_name, x11_darkname))
LYXERR0("Bad lyxrc set_color for " << lyx_name);
- LYXERR(Debug::LYXRC, "Set " << lyx_name << "(" << col << ") to " << x11_name);
+ LYXERR(Debug::LYXRC, "Set " << lyx_name << "(" << col << ") to "
+ << x11_name << " and " << x11_darkname);
break;
}
@@ -1987,12 +1992,15 @@ void LyXRC::write(ostream & os, bool ignore_system_lyxrc, string const & name) c
case RC_SET_COLOR:
for (int i = 0; i < Color_ignore; ++i) {
ColorCode lc = static_cast<ColorCode>(i);
- string const col = lcolor.getX11HexName(lc);
+ string const col = lcolor.getAllX11HexNames(lc).first;
+ string const darkcol = lcolor.getAllX11HexNames(lc).second;
if (ignore_system_lyxrc
- || col != system_lcolor.getX11HexName(lc)) {
+ || col != system_lcolor.getAllX11HexNames(lc).first
+ || darkcol != system_lcolor.getAllX11HexNames(lc).second) {
os << "\\set_color \""
<< lcolor.getLyXName(lc) << "\" \""
- << col << "\"\n";
+ << col << "\" \""
+ << darkcol << "\"\n";
}
}
if (tag != RC_LAST)
diff --git a/src/frontends/qt/ColorCache.cpp b/src/frontends/qt/ColorCache.cpp
index 1b87641..822f44a 100644
--- a/src/frontends/qt/ColorCache.cpp
+++ b/src/frontends/qt/ColorCache.cpp
@@ -62,7 +62,7 @@ QPalette::ColorRole role(ColorCode col)
void ColorCache::init()
{
for (int col = 0; col <= Color_ignore; ++col) {
- lcolors_[col] = QColor(lcolor.getX11HexName(ColorCode(col)).c_str());
+ lcolors_[col] = QColor(lcolor.getX11HexName(ColorCode(col), isDarkMode()).c_str());
}
initialized_ = true;
@@ -104,7 +104,7 @@ QColor ColorCache::get(Color const & color, bool syscolors) const
(base_color.blue() + merge_color.blue()) / 2);
}
// used by branches
- return QColor(lcolor.getX11HexName(color.baseColor).c_str());
+ return QColor(lcolor.getX11HexName(color.baseColor, isDarkMode()).c_str());
}
@@ -119,6 +119,16 @@ bool ColorCache::isSystem(ColorCode const color) const
}
+bool ColorCache::isDarkMode() const
+{
+ QPalette palette = QPalette();
+ QColor text_color = palette.color(QPalette::Active, QPalette::WindowText);
+ QColor bg_color = palette.color(QPalette::Active, QPalette::Window);
+
+ return (text_color.black() < bg_color.black());
+}
+
+
QColor const rgb2qcolor(RGBColor const & rgb)
{
return QColor(rgb.r, rgb.g, rgb.b);
diff --git a/src/frontends/qt/ColorCache.h b/src/frontends/qt/ColorCache.h
index beb680f..09a995d 100644
--- a/src/frontends/qt/ColorCache.h
+++ b/src/frontends/qt/ColorCache.h
@@ -39,6 +39,9 @@ public:
/// is this color replaced when LyXRC::use_system_color is true?
bool isSystem(ColorCode color) const;
+ /// guess whether we are in dark mode
+ bool isDarkMode() const;
+
/// change the undelying palette
void setPalette(QPalette const pal) { pal_ = pal; clear(); }
diff --git a/src/frontends/qt/GuiApplication.cpp b/src/frontends/qt/GuiApplication.cpp
index 8e4d48a..a4c9132 100644
--- a/src/frontends/qt/GuiApplication.cpp
+++ b/src/frontends/qt/GuiApplication.cpp
@@ -1807,11 +1807,12 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
case LFUN_SET_COLOR: {
string const lyx_name = cmd.getArg(0);
- string const x11_name = cmd.getArg(1);
+ string x11_name = cmd.getArg(1);
+ string x11_darkname = cmd.getArg(2);
if (lyx_name.empty() || x11_name.empty()) {
if (current_view_)
current_view_->message(
- _("Syntax: set-color <lyx_name> <x11_name>"));
+ _("Syntax: set-color <lyx_name> <x11_name> <x11_darkname>"));
break;
}
@@ -1824,7 +1825,11 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
graphics::GCache::get().changeDisplay(true);
#endif
- if (!lcolor.setColor(lyx_name, x11_name)) {
+ if (x11_darkname.empty() && colorCache().isDarkMode()) {
+ x11_darkname = x11_name;
+ x11_name.clear();
+ }
+ if (!lcolor.setColor(lyx_name, x11_name, x11_darkname)) {
if (current_view_)
current_view_->message(
bformat(_("Set-color \"%1$s\" failed "
diff --git a/src/frontends/qt/GuiPrefs.cpp b/src/frontends/qt/GuiPrefs.cpp
index beb424f..b4af558 100644
--- a/src/frontends/qt/GuiPrefs.cpp
+++ b/src/frontends/qt/GuiPrefs.cpp
@@ -1251,7 +1251,8 @@ bool PrefColors::isDefaultColor(int const row, QString const & color)
QColor PrefColors::getDefaultColorByRow(int const row)
{
ColorSet const defaultcolor;
- return defaultcolor.getX11HexName(lcolors_[size_t(row)]).c_str();
+ return defaultcolor.getX11HexName(lcolors_[size_t(row)],
+ guiApp->colorCache().isDarkMode()).c_str();
}
More information about the lyx-cvs
mailing list