Yehowshua/ZIm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "theme: Turn ThemeRegistry into a trait (#20076)" (#20094)

Browse source 
This PR reverts #20076 to turn the `ThemeRegistry` back into a regular
struct again.

It doesn't actually help us by having it behind a trait.

Release Notes:

- N/A
This commit is contained in:
Marshall Bowers 2024-11-01 15:34:20 -04:00 • committed by GitHub
parent 4084ba36f9
commit 273173ec8a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
12 changed files with 89 additions and 151 deletions
Download patch file Download diff file Expand all files Collapse all files

1
crates/theme/Cargo.toml
View file

@ -18,7 +18,6 @@ doctest = false
[dependencies]
anyhow.workspace = true
async-trait.workspace = true
collections.workspace = true
derive_more.workspace = true
fs.workspace = true

189
crates/theme/src/registry.rs
View file

@ -1,8 +1,7 @@
use std::sync::Arc;
use std::{fmt::Debug, path::Path};
use anyhow::{anyhow, bail, Context, Result};
use async_trait::async_trait;
use anyhow::{anyhow, Context, Result};
use collections::HashMap;
use derive_more::{Deref, DerefMut};
use fs::Fs;
@ -30,34 +29,22 @@ pub struct ThemeMeta {
/// inserting the [`ThemeRegistry`] into the context as a global.
///
/// This should not be exposed outside of this module.
#[derive(Deref, DerefMut)]
struct GlobalThemeRegistry(Arc<dyn ThemeRegistry>);
#[derive(Default, Deref, DerefMut)]
struct GlobalThemeRegistry(Arc<ThemeRegistry>);
impl Global for GlobalThemeRegistry {}
/// A registry for themes.
#[async_trait]
pub trait ThemeRegistry: Send + Sync + 'static {
/// Returns the names of all themes in the registry.
fn list_names(&self) -> Vec<SharedString>;
/// Returns the metadata of all themes in the registry.
fn list(&self) -> Vec<ThemeMeta>;
/// Returns the theme with the given name.
fn get(&self, name: &str) -> Result<Arc<Theme>>;
/// Loads the user theme from the specified path and adds it to the registry.
async fn load_user_theme(&self, theme_path: &Path, fs: Arc<dyn Fs>) -> Result<()>;
/// Loads the user themes from the specified directory and adds them to the registry.
async fn load_user_themes(&self, themes_path: &Path, fs: Arc<dyn Fs>) -> Result<()>;
/// Removes the themes with the given names from the registry.
fn remove_user_themes(&self, themes_to_remove: &[SharedString]);
struct ThemeRegistryState {
themes: HashMap<SharedString, Arc<Theme>>,
}
impl dyn ThemeRegistry {
/// The registry for themes.
pub struct ThemeRegistry {
state: RwLock<ThemeRegistryState>,
assets: Box<dyn AssetSource>,
}
impl ThemeRegistry {
/// Returns the global [`ThemeRegistry`].
pub fn global(cx: &AppContext) -> Arc<Self> {
cx.global::<GlobalThemeRegistry>().0.clone()
@ -67,37 +54,18 @@ impl dyn ThemeRegistry {
///
/// Inserts a default [`ThemeRegistry`] if one does not yet exist.
pub fn default_global(cx: &mut AppContext) -> Arc<Self> {
if let Some(registry) = cx.try_global::<GlobalThemeRegistry>() {
return registry.0.clone();
}
let registry = Arc::new(RealThemeRegistry::default());
cx.set_global(GlobalThemeRegistry(registry.clone()));
registry
cx.default_global::<GlobalThemeRegistry>().0.clone()
}
}
struct RealThemeRegistryState {
themes: HashMap<SharedString, Arc<Theme>>,
}
/// The registry for themes.
pub struct RealThemeRegistry {
state: RwLock<RealThemeRegistryState>,
assets: Box<dyn AssetSource>,
}
impl RealThemeRegistry {
/// Sets the global [`ThemeRegistry`].
pub(crate) fn set_global(self: Arc<Self>, cx: &mut AppContext) {
cx.set_global(GlobalThemeRegistry(self));
pub(crate) fn set_global(assets: Box<dyn AssetSource>, cx: &mut AppContext) {
cx.set_global(GlobalThemeRegistry(Arc::new(ThemeRegistry::new(assets))));
}
/// Creates a new [`ThemeRegistry`] with the given [`AssetSource`].
pub fn new(assets: Box<dyn AssetSource>) -> Self {
let registry = Self {
state: RwLock::new(RealThemeRegistryState {
state: RwLock::new(ThemeRegistryState {
themes: HashMap::default(),
}),
assets,
@ -132,11 +100,49 @@ impl RealThemeRegistry {
}
}
/// Removes the themes with the given names from the registry.
pub fn remove_user_themes(&self, themes_to_remove: &[SharedString]) {
self.state
.write()
.themes
.retain(|name, _| !themes_to_remove.contains(name))
}
/// Removes all themes from the registry.
pub fn clear(&self) {
self.state.write().themes.clear();
}
/// Returns the names of all themes in the registry.
pub fn list_names(&self) -> Vec<SharedString> {
let mut names = self.state.read().themes.keys().cloned().collect::<Vec<_>>();
names.sort();
names
}
/// Returns the metadata of all themes in the registry.
pub fn list(&self) -> Vec<ThemeMeta> {
self.state
.read()
.themes
.values()
.map(|theme| ThemeMeta {
name: theme.name.clone(),
appearance: theme.appearance(),
})
.collect()
}
/// Returns the theme with the given name.
pub fn get(&self, name: &str) -> Result<Arc<Theme>> {
self.state
.read()
.themes
.get(name)
.ok_or_else(|| anyhow!("theme not found: {}", name))
.cloned()
}
/// Loads the themes bundled with the Zed binary and adds them to the registry.
pub fn load_bundled_themes(&self) {
let theme_paths = self
@ -161,52 +167,9 @@ impl RealThemeRegistry {
self.insert_user_theme_families([theme_family]);
}
}
}
impl Default for RealThemeRegistry {
fn default() -> Self {
Self::new(Box::new(()))
}
}
#[async_trait]
impl ThemeRegistry for RealThemeRegistry {
fn list_names(&self) -> Vec<SharedString> {
let mut names = self.state.read().themes.keys().cloned().collect::<Vec<_>>();
names.sort();
names
}
fn list(&self) -> Vec<ThemeMeta> {
self.state
.read()
.themes
.values()
.map(|theme| ThemeMeta {
name: theme.name.clone(),
appearance: theme.appearance(),
})
.collect()
}
fn get(&self, name: &str) -> Result<Arc<Theme>> {
self.state
.read()
.themes
.get(name)
.ok_or_else(|| anyhow!("theme not found: {}", name))
.cloned()
}
async fn load_user_theme(&self, theme_path: &Path, fs: Arc<dyn Fs>) -> Result<()> {
let theme = read_user_theme(theme_path, fs).await?;
self.insert_user_theme_families([theme]);
Ok(())
}
async fn load_user_themes(&self, themes_path: &Path, fs: Arc<dyn Fs>) -> Result<()> {
/// Loads the user themes from the specified directory and adds them to the registry.
pub async fn load_user_themes(&self, themes_path: &Path, fs: Arc<dyn Fs>) -> Result<()> {
let mut theme_paths = fs
.read_dir(themes_path)
.await
@ -225,38 +188,18 @@ impl ThemeRegistry for RealThemeRegistry {
Ok(())
}
fn remove_user_themes(&self, themes_to_remove: &[SharedString]) {
self.state
.write()
.themes
.retain(|name, _| !themes_to_remove.contains(name))
/// Loads the user theme from the specified path and adds it to the registry.
pub async fn load_user_theme(&self, theme_path: &Path, fs: Arc<dyn Fs>) -> Result<()> {
let theme = read_user_theme(theme_path, fs).await?;
self.insert_user_theme_families([theme]);
Ok(())
}
}
/// A theme registry that doesn't have any behavior.
pub struct VoidThemeRegistry;
#[async_trait]
impl ThemeRegistry for VoidThemeRegistry {
fn list_names(&self) -> Vec<SharedString> {
Vec::new()
impl Default for ThemeRegistry {
fn default() -> Self {
Self::new(Box::new(()))
}
fn list(&self) -> Vec<ThemeMeta> {
Vec::new()
}
fn get(&self, name: &str) -> Result<Arc<Theme>> {
bail!("cannot retrieve theme {name:?} from a void theme registry")
}
async fn load_user_theme(&self, _theme_path: &Path, _fs: Arc<dyn Fs>) -> Result<()> {
Ok(())
}
async fn load_user_themes(&self, _themes_path: &Path, _fs: Arc<dyn Fs>) -> Result<()> {
Ok(())
}
fn remove_user_themes(&self, _themes_to_remove: &[SharedString]) {}
}

8
crates/theme/src/settings.rs
View file

@ -150,7 +150,7 @@ impl ThemeSettings {
// If the selected theme doesn't exist, fall back to a default theme
// based on the system appearance.
let theme_registry = <dyn ThemeRegistry>::global(cx);
let theme_registry = ThemeRegistry::global(cx);
if theme_registry.get(theme_name).ok().is_none() {
theme_name = Self::default_theme(*system_appearance);
};
@ -446,7 +446,7 @@ impl ThemeSettings {
/// Returns a `Some` containing the new theme if it was successful.
/// Returns `None` otherwise.
pub fn switch_theme(&mut self, theme: &str, cx: &mut AppContext) -> Option<Arc<Theme>> {
let themes = <dyn ThemeRegistry>::default_global(cx);
let themes = ThemeRegistry::default_global(cx);
let mut new_theme = None;
@ -598,7 +598,7 @@ impl settings::Settings for ThemeSettings {
type FileContent = ThemeSettingsContent;
fn load(sources: SettingsSources<Self::FileContent>, cx: &mut AppContext) -> Result<Self> {
let themes = <dyn ThemeRegistry>::default_global(cx);
let themes = ThemeRegistry::default_global(cx);
let system_appearance = SystemAppearance::default_global(cx);
let defaults = sources.default;
@ -710,7 +710,7 @@ impl settings::Settings for ThemeSettings {
cx: &AppContext,
) -> schemars::schema::RootSchema {
let mut root_schema = generator.root_schema_for::<ThemeSettingsContent>();
let theme_names = <dyn ThemeRegistry>::global(cx)
let theme_names = ThemeRegistry::global(cx)
.list_names()
.into_iter()
.map(|theme_name| Value::String(theme_name.to_string()))

5
crates/theme/src/theme.rs
View file

@ -88,11 +88,10 @@ pub fn init(themes_to_load: LoadThemes, cx: &mut AppContext) {
LoadThemes::JustBase => (Box::new(()) as Box<dyn AssetSource>, false),
LoadThemes::All(assets) => (assets, true),
};
let registry = Arc::new(RealThemeRegistry::new(assets));
registry.clone().set_global(cx);
ThemeRegistry::set_global(assets, cx);
if load_user_themes {
registry.load_bundled_themes();
ThemeRegistry::global(cx).load_bundled_themes();
}
ThemeSettings::register(cx);