Apply suggestions from code review

Co-authored-by: Danielle Foré <danielle@elementary.io>
elementary · Sep 20, 2024 · a428914 · a428914
1 parent 432866c
commit a428914
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/demo/Views/StyleManagerView.vala b/demo/Views/StyleManagerView.vala
@@ -5,17 +5,17 @@
 
 public class StyleManagerView : Gtk.Box {
  construct {
- var label = new Gtk.Label (_("Override the application theme:"));
+ var label = new Granite.HeaderLabel ("Visual Style");
 
- var dont_button = new Gtk.CheckButton.with_label (_("Use system theme")) {
+ var dont_button = new Gtk.CheckButton.with_label ("Follow system setting") {
  active = true
  };
 
- var force_light = new Gtk.CheckButton.with_label (_("Force light theme")) {
+ var force_light = new Gtk.CheckButton.with_label ("Light") {
  group = dont_button
  };
 
- var force_dark = new Gtk.CheckButton.with_label (_("Force dark theme")) {
+ var force_dark = new Gtk.CheckButton.with_label ("Dark") {
  group = force_light
  };
 

diff --git a/lib/StyleManager.vala b/lib/StyleManager.vala
@@ -31,14 +31,14 @@ public class Granite.StyleManager : Object {
  }
 
  /**
- * If this is set to NO_PREFERENCE the systems preferred color scheme will be used.
- * Otherwise the color scheme set here will be used.
- * Default is NO_PREFERENCE.
+ * The {@link Granite.Settings.ColorScheme} requested by the application
+ * Uses value from {@link Granite.Settings.prefers_color_scheme} when set to {@link Granite.Settings.ColorScheme.NO_PREFERENCE }.
+ * Default value is {@link Granite.Settings.ColorScheme.NO_PREFERENCE }
  */
  public Settings.ColorScheme color_scheme_override { get; set; default = NO_PREFERENCE; }
 
  /**
- * The Gdk.Display this StyleManager handles.
+ * The {@link Gdk.Display} handled by #this.
  */
  public Gdk.Display display { get; construct; }