Skip to content

Commit

Permalink
revert dark mode breaking change
Browse files Browse the repository at this point in the history
  • Loading branch information
themsaid committed Apr 2, 2020
1 parent 0cde593 commit fdfbd5b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 28 deletions.
15 changes: 0 additions & 15 deletions config/horizon.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,21 +4,6 @@

return [

/*
|--------------------------------------------------------------------------
| Theme
|--------------------------------------------------------------------------
|
| Here you can define which theme Horizon should load. Horizon currently
| provides a light theme and a dark theme, and you may provide one of
| those two values in order to determine which theme will get used.
|
| Supported: "light", "dark"
|
*/

'theme' => 'light',

/*
|--------------------------------------------------------------------------
| Horizon Domain
Expand Down
15 changes: 2 additions & 13 deletions src/Horizon.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,16 +61,6 @@ class Horizon
'Metrics', 'Locks', 'Processes',
];

/**
* Create a new Horizon configuration instance.
*
* @return void
*/
public function __construct()
{
static::$useDarkTheme = config('horizon.theme') === 'dark';
}

/**
* Determine if the given request can access the Horizon dashboard.
*
Expand Down Expand Up @@ -120,12 +110,11 @@ public static function use($connection)
/**
* Specifies that Horizon should use the dark theme.
*
* @param bool $on
* @return static
*/
public static function night($on = true)
public static function night()
{
static::$useDarkTheme = $on;
static::$useDarkTheme = true;

return new static;
}
Expand Down

3 comments on commit fdfbd5b

@francoism90
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@themsaid Dark mode isn't working at all now, will it return back?

@themsaid
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@francoism90 upgrade to the latest release.

@francoism90
Copy link

@francoism90 francoism90 commented on fdfbd5b Apr 3, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@themsaid I'm already running: laravel/horizon v4.2.1 - do you mean the upcoming version?

Please sign in to comment.