fixed style mistakes, removed unnecessary legend code and fixed load_exam in app.py

Author: magdalena-grubesa

src/my_studysmart.xlsx

@@ -130,10 +130,7 @@ def build_chart(schedule, spaced, color_map):
     """Build Plotly bar chart from schedule DataFrame."""
     schedule_plot = schedule.copy()
     schedule_plot["date"] = schedule_plot["date"].apply(str)
-    # strip exam prefix for display in chart legend
-    schedule_plot["display_subject"] = schedule_plot["subject"].apply(
-        lambda s: s.split(": ", 1)[-1] if ": " in s else s
-    )
+    
     return px.bar(
         schedule_plot,
         x="date",
@@ -169,29 +166,6 @@ def build_weekly_view(schedule, spaced, week_offset=0, color_map=None, exam_colo
     week_start = week_start + timedelta(weeks=week_offset)
     week_end = week_start + timedelta(days=6)
 
-    # build legend
-    legend_items = []
-    for exam_name, color in exam_colors.items():
-        legend_items.append(
-            html.Span([
-                html.Span(style={
-                    "display": "inline-block",
-                    "width": "12px",
-                    "height": "12px",
-                    "borderRadius": "3px",
-                    "backgroundColor": color,
-                    "marginRight": "4px",
-                    "verticalAlign": "middle"
-                }),
-                html.Span(exam_name, style={
-                    "fontSize": "12px",
-                    "marginRight": "16px",
-                    "color": "var(--color-text-secondary)"
-                })
-            ])
-        )
-    legend = html.Div(legend_items, style={"marginBottom": "12px"})
-
     # build week days
     day_cols = []
     for i in range(7):
@@ -257,7 +231,7 @@ def build_weekly_view(schedule, spaced, week_offset=0, color_map=None, exam_colo
         "overflow": "hidden"
     })
 
-    return html.Div([nav, legend, calendar])
+    return html.Div([nav,calendar])
 
 
 def build_tips(schedule, exam_dates, start, default_hours):
@@ -423,6 +397,10 @@ def load_exams(n_clicks):
         df["date"] = pd.to_datetime(df["date"]).dt.date
 
         for _, row in df.iterrows():
+            # skip if this exam name is already loaded
+            if any(e["name"] == row["name"] for e in exams):
+                continue
+
             topic_list = []
             if pd.notna(row["topics"]) and row["topics"]:
                 for t in row["topics"].split(","):

src/study_smart/app.py

@@ -36,14 +36,14 @@ def _schedule_reviews(topic_name, first_study_day, exam_date,
     for interval in review_intervals:
         review_date = first_study_day + timedelta(days=interval)
 
-        #skip if review date is on or after exam date
+        # skip if review date is on or after exam date
         if review_date >= exam_date:
             continue
-        #check remaining hours on review date
+        # check remaining hours on review date
         available = _available_hours_per_day(review_date, commitments=commitments)
         remaining = available - allocated.get(review_date, 0.0)
 
-        #no room - skip this review 
+        # no room - skip this review 
         if remaining <= 0:
             continue
 

src/study_smart/schedule.py

@@ -4,7 +4,6 @@
 Tests cover:
 - _available_hours_per_day: commitment deduction and minimum floor
 - _round_to_half: rounding behaviour
-- _distribute_hours: proportional distribution and edge cases
 - build_schedule: single and multiple exams, topics, commitments, overload handling
 - build_schedule with spaced repetition: review intervals, skipping, topic defaults
 - generate_tips: active recall, deep understanding, and light load tips