Merge pull request #1968 from zed-industries/fix-text-size-in-updates

Mikayla Maki created 3 years ago

Fix text size in notifications

Change summary

crates/auto_update/src/update_notification.rs     | 2 +-
crates/workspace/src/notifications.rs             | 2 +-
styles/src/styleTree/simpleMessageNotification.ts | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)

Detailed changes

crates/auto_update/src/update_notification.rs

@@ -28,7 +28,7 @@ impl View for UpdateNotification {
 
     fn render(&mut self, cx: &mut gpui::RenderContext<'_, Self>) -> gpui::ElementBox {
         let theme = cx.global::<Settings>().theme.clone();
-        let theme = &theme.simple_message_notification;
+        let theme = &theme.update_notification;
 
         let app_name = cx.global::<ReleaseChannel>().display_name();
 

crates/workspace/src/notifications.rs

@@ -206,7 +206,7 @@ pub mod simple_message_notification {
 
         fn render(&mut self, cx: &mut gpui::RenderContext<'_, Self>) -> gpui::ElementBox {
             let theme = cx.global::<Settings>().theme.clone();
-            let theme = &theme.update_notification;
+            let theme = &theme.simple_message_notification;
 
             enum MessageNotificationTag {}
 

styles/src/styleTree/simpleMessageNotification.ts

@@ -7,11 +7,11 @@ export default function simpleMessageNotification(colorScheme: ColorScheme): Obj
   let layer = colorScheme.middle;
   return {
     message: {
-      ...text(layer, "sans", { size: "md" }),
+      ...text(layer, "sans", { size: "xs" }),
       margin: { left: headerPadding, right: headerPadding },
     },
     actionMessage: {
-      ...text(layer, "sans", { size: "md" }),
+      ...text(layer, "sans", { size: "xs" }),
       margin: { left: headerPadding, top: 6, bottom: 6 },
       hover: {
         color: foreground(layer, "hovered"),