From 5dc87e0ffc9163689f5f46aa31893002cc8de02f Mon Sep 17 00:00:00 2001
From: vincentinttsh <vincentinttsh@gmail.com>
Date: Sun, 24 Jan 2021 18:14:37 +0800
Subject: [PATCH 1/2] Course Detail & use course number to delete

---
 js/components/coursesList.vue | 4 ++--
 js/components/main.vue        | 7 ++++---
 js/components/table.vue       | 3 ++-
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/js/components/coursesList.vue b/js/components/coursesList.vue
index f576c3b..0397596 100644
--- a/js/components/coursesList.vue
+++ b/js/components/coursesList.vue
@@ -74,7 +74,7 @@ var coursesList = {
 						<td>
 							<div class="container row py-2 px-0">
 								<div class="col-12 pr-1">
-									<b>{{ course.name }} (<a v-bind:href="course.link">詳</a>)</b>
+									<b>{{ course.name }} (<a v-bind:href="course.link" target="_blank">詳</a>)</b>
 									—— {{ (course.department.indexOf(', ')!=-1) ?(course.department.split(', ')[1]) :(course.department) }}
 								</div>
 								<div class="col-sm-8 pr-1">
@@ -96,7 +96,7 @@ var coursesList = {
 						<td>
                         <div class="container row py-2 px-0">
                             <div class="col-12 pr-1">
-                                <b>{{ course.name }} (<a v-bind:href="course.link">詳</a>)</b>
+                                <b>{{ course.name }} (<a v-bind:href="course.link" target="_blank">詳</a>)</b>
                             </div>
                             <div class="col-sm-8 pr-1">
                                 {{ course.teacher }} ‧ {{ course.time }} 
diff --git a/js/components/main.vue b/js/components/main.vue
index abe0332..843450e 100644
--- a/js/components/main.vue
+++ b/js/components/main.vue
@@ -161,14 +161,15 @@ var mainWindow = {
                 this.selectCourses.push({
                     'time': t,
                     'name': course.name,
-                    'temp': false
+                    'temp': false,
+                    'number': course.number
                 })
             }
         },
         'removeCourse': function (course) {
-            console.log("remove " + course)
+            console.log("remove " + course.name)
             for (var i = this.selectCourses.length - 1; i >= 0; i--) {
-                if (this.selectCourses[i].name == course) {
+                if (this.selectCourses[i].number == course.number) {
                     this.selectCourses.splice(i, 1)
                 }
             }
diff --git a/js/components/table.vue b/js/components/table.vue
index 999c6b1..68f1495 100644
--- a/js/components/table.vue
+++ b/js/components/table.vue
@@ -5,7 +5,7 @@ var courseDiv = {
             {{ course.name }}
             <button type="button" 
                 v-if="!is_shared"
-                v-on:click="$emit('remove-course', course.name)"
+                v-on:click="$emit('remove-course', course)"
                 class="btn btn-danger btn-sm"
                 :style="{'display': is_print ? 'none' : 'inline-block'}"
             >
@@ -52,6 +52,7 @@ var courseTable = {
             for(var c of this.select_c){
                 this.courses[c.time] = {
                     'name': c.name,
+                    'number': c.number,
                     'temp': c.temp
                 }
 

From d161c10647906403c0aadc5e48b66e32b5e9e409 Mon Sep 17 00:00:00 2001
From: vincentinttsh <vincentinttsh@gmail.com>
Date: Sun, 24 Jan 2021 20:13:27 +0800
Subject: [PATCH 2/2] use course number and class to delete

---
 js/components/main.vue  | 5 +++--
 js/components/table.vue | 1 +
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/js/components/main.vue b/js/components/main.vue
index 843450e..46557bd 100644
--- a/js/components/main.vue
+++ b/js/components/main.vue
@@ -162,14 +162,15 @@ var mainWindow = {
                     'time': t,
                     'name': course.name,
                     'temp': false,
-                    'number': course.number
+                    'number': course.number,
+                    'class': course.class
                 })
             }
         },
         'removeCourse': function (course) {
             console.log("remove " + course.name)
             for (var i = this.selectCourses.length - 1; i >= 0; i--) {
-                if (this.selectCourses[i].number == course.number) {
+                if (this.selectCourses[i].number === course.number && this.selectCourses[i].class === course.class) {
                     this.selectCourses.splice(i, 1)
                 }
             }
diff --git a/js/components/table.vue b/js/components/table.vue
index 68f1495..475e848 100644
--- a/js/components/table.vue
+++ b/js/components/table.vue
@@ -53,6 +53,7 @@ var courseTable = {
                 this.courses[c.time] = {
                     'name': c.name,
                     'number': c.number,
+                    'class': c.class,
                     'temp': c.temp
                 }