From 2382209594ee51996fbe9caf409b39ff4a5b62f4 Mon Sep 17 00:00:00 2001 From: sloumdrone Date: Sun, 4 Nov 2018 17:20:51 -0800 Subject: [PATCH 1/5] Added loading bar. It is finicky and not animating properly --- gui.py | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/gui.py b/gui.py index e05367d..fbd3a1e 100644 --- a/gui.py +++ b/gui.py @@ -1,6 +1,7 @@ #!/usr/bin/env python3 import tkinter as tk +from tkinter import ttk import tkinter.simpledialog as dialog from connect import connect as conn from parser import parser @@ -72,6 +73,7 @@ class GUI: #status bar objects self.status_info = tk.Label(self.status_bar, textvariable=self.message_bar_content, bg=self.STATUS_BG, takefocus=0, fg=self.ACTIVELINK) + self.pack_geometry() self.add_status_titles() self.add_event_listeners() @@ -143,6 +145,11 @@ class GUI: # ------------Start navigation methods---------------------------- def handle_request(self,event=False, url=False, history=True): + self.progress_bar = ttk.Progressbar(self.entry_url, orient="horizontal", length=130, mode='indeterminate') + self.progress_bar.pack(side=tk.RIGHT, padx=(0,10)) + self.progress_bar.start(10) + self.progress_bar.update_idletasks() + url = url if url else self.entry_url.get() parsed_url = self.parse_url(url) @@ -268,7 +275,7 @@ class GUI: entry = '{}{}\t{}\t{}\t{}\n'.format(url['type'], x['name'], url['resource'], url['host'], url['port']) header += entry return header - # self.send_to_screen(data=header, clear=False) + def show_search(self): text1 = ' __ ___ __ __\n/__` |__ /\ |__) / ` |__|\n.__/ |___ /~~\ | \ \__, | |\n\n\nPlease enter your search terms and press the enter key:\n\n' @@ -279,6 +286,7 @@ class GUI: self.site_display.insert(tk.END,text1) self.site_display.window_create(tk.END,window=self.search) self.site_display.config(state=tk.DISABLED) + self.search.focus_set() def query_search_engine(self, event): @@ -291,7 +299,6 @@ class GUI: self.search = None - def show_menu(self, data, clear=True): if not data: #error handling will go here @@ -349,7 +356,6 @@ class GUI: link_count += 1 self.site_display.config(state=tk.DISABLED) - return True @@ -379,6 +385,10 @@ class GUI: elif itemtype in ['p','I','g']: self.show_image(data) + try: + self.progress_bar.destroy() + except: + pass def update_status(self, event, href=False): if href: From 74c1ad4ca8205c6757500e79de96706f2c5f76d4 Mon Sep 17 00:00:00 2001 From: sloumdrone Date: Sun, 4 Nov 2018 21:30:33 -0800 Subject: [PATCH 2/5] Removed progress bar in favor of loading widget in the address bar --- gui.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/gui.py b/gui.py index fbd3a1e..d01f98a 100644 --- a/gui.py +++ b/gui.py @@ -1,7 +1,6 @@ #!/usr/bin/env python3 import tkinter as tk -from tkinter import ttk import tkinter.simpledialog as dialog from connect import connect as conn from parser import parser @@ -145,9 +144,8 @@ class GUI: # ------------Start navigation methods---------------------------- def handle_request(self,event=False, url=False, history=True): - self.progress_bar = ttk.Progressbar(self.entry_url, orient="horizontal", length=130, mode='indeterminate') + self.progress_bar = tk.Label(self.entry_url, text=' Loading... ', width=12, relief=tk.FLAT, height=1, fg='#FFFFFF', bg=self.TYPES) self.progress_bar.pack(side=tk.RIGHT, padx=(0,10)) - self.progress_bar.start(10) self.progress_bar.update_idletasks() url = url if url else self.entry_url.get() @@ -175,6 +173,7 @@ class GUI: self.send_to_screen(self.conn.raw_response,self.conn.filetype) + def parse_url(self, url=False): parsed_url = self.parser.parse_url(url) From ccb66be447257b2f4b25488a4c1fe5bd258903c5 Mon Sep 17 00:00:00 2001 From: sloumdrone Date: Sun, 4 Nov 2018 22:04:15 -0800 Subject: [PATCH 3/5] Fixed removal of terminal period in text files --- connect.py | 4 ++-- gui.py | 23 ++++++++++++++--------- 2 files changed, 16 insertions(+), 11 deletions(-) diff --git a/connect.py b/connect.py index bfa108a..b07df7f 100644 --- a/connect.py +++ b/connect.py @@ -21,11 +21,11 @@ class connect: except socket.timeout: print('Socket timeout') socket_conn.close() - return {'type': '3', 'body': '3ERROR: Server request timed out\tfalse\tnull.host\t1'} + return {'type': '1', 'body': '3ERROR: Server request timed out\tfalse\tnull.host\t1'} except Exception as e: print('Misc socket error: ', e) socket_conn.close() - return {'type': '3', 'body': '3ERROR: Unable to communicate with remote server\tfalse\tnull.host\t1'} + return {'type': '1', 'body': '3ERROR: Unable to communicate with remote server\tfalse\tnull.host\t1'} try: self.raw_response = response.read() diff --git a/gui.py b/gui.py index d01f98a..6d1cc82 100644 --- a/gui.py +++ b/gui.py @@ -144,9 +144,9 @@ class GUI: # ------------Start navigation methods---------------------------- def handle_request(self,event=False, url=False, history=True): - self.progress_bar = tk.Label(self.entry_url, text=' Loading... ', width=12, relief=tk.FLAT, height=1, fg='#FFFFFF', bg=self.TYPES) - self.progress_bar.pack(side=tk.RIGHT, padx=(0,10)) - self.progress_bar.update_idletasks() + self.loading_bar = tk.Label(self.entry_url, text=' Loading... ', width=12, relief=tk.FLAT, height=1, fg='#FFFFFF', bg=self.TYPES) + self.loading_bar.pack(side=tk.RIGHT, padx=(0,10)) + self.loading_bar.update_idletasks() url = url if url else self.entry_url.get() parsed_url = self.parse_url(url) @@ -170,7 +170,7 @@ class GUI: if not data: return False #error handling goes here - self.send_to_screen(self.conn.raw_response,self.conn.filetype) + self.send_to_screen(data['body'],data['type']) @@ -197,8 +197,7 @@ class GUI: self.site_display.focus_set() self.config["last_viewed"] = url - self.send_to_screen(self.conn.raw_response, self.conn.filetype) - return True + return response def add_to_history(self, url): @@ -287,6 +286,12 @@ class GUI: self.site_display.config(state=tk.DISABLED) self.search.focus_set() + try: + self.loading_bar.destroy() + except: + pass + + def query_search_engine(self, event): base_url = self.entry_url.get() @@ -324,7 +329,7 @@ class GUI: link_count = 0 - for x in data[1:]: + for x in data: if x['type'] == 'i': self.site_display.insert(tk.END,' \t\t{}\n'.format(x['description'])) elif x['type'] == '3': @@ -359,7 +364,7 @@ class GUI: def show_text(self, data): - if data[-2:] == '\n.': + if data[-2:] == '.\n': data = data[:-2] self.site_display.config(state=tk.NORMAL) self.site_display.delete(1.0, tk.END) @@ -385,7 +390,7 @@ class GUI: self.show_image(data) try: - self.progress_bar.destroy() + self.loading_bar.destroy() except: pass From 0d69c834eeb48097e2477f2d4156e9704b727576 Mon Sep 17 00:00:00 2001 From: sloumdrone Date: Wed, 7 Nov 2018 23:04:34 -0800 Subject: [PATCH 4/5] Added ability to delete favorite via r click --- gui.py | 42 +++++++++++++++++++++++++++++++++--------- 1 file changed, 33 insertions(+), 9 deletions(-) diff --git a/gui.py b/gui.py index 6d1cc82..ecbfc42 100644 --- a/gui.py +++ b/gui.py @@ -45,7 +45,11 @@ class GUI: #create and configure root window self.root = tk.Tk(className='Burrow') self.root.title('Burrow') - self.root.geometry("1200x800") + sh = self.root.winfo_screenheight() + sw = self.root.winfo_screenwidth() + w = int(sw * 0.7) + h = sh - 200 + self.root.geometry("{}x{}+{}+{}".format(w, h, sw//2-w//2, 50)) self.add_assets() #main frame objects @@ -227,8 +231,9 @@ class GUI: self.entry_url.insert(tk.END, 'home') if event: self.add_to_history('home') - data += self.load_favorites() - self.send_to_screen(data, '1') + data2 = self.load_favorites() + link_count = self.send_to_screen(data, '1', True) + self.send_to_screen(data2, '1', False, link_count) def go_back(self, event): @@ -303,7 +308,7 @@ class GUI: self.search = None - def show_menu(self, data, clear=True): + def show_menu(self, data, clear=True, links=0): if not data: #error handling will go here return False @@ -327,7 +332,7 @@ class GUI: if clear: self.site_display.delete(1.0, tk.END) - link_count = 0 + link_count = links for x in data: if x['type'] == 'i': @@ -349,18 +354,25 @@ class GUI: tag_name = 'link{}'.format(link_count) callback = (lambda event, href=link, tag_name=tag_name: self.gotolink(event, href, tag_name)) hover = (lambda event, href=link, tag_name=tag_name: self.hoverlink(event, href, tag_name)) + favorite = [x for x in self.config['favorites'] if x['url'] == link] clear = (lambda event, tag_name=tag_name: self.clear_status(event, tag_name)) self.site_display.tag_bind(tag_name, "", callback) self.site_display.tag_bind(tag_name, "", hover) self.site_display.tag_bind(tag_name, '', clear) self.site_display.insert(tk.END, types[x['type']], ('type_tag',)) self.site_display.insert(tk.END,'\t\t') - self.site_display.insert(tk.END, x['description'], (tag_name,'linkcolor')) + if favorite: + tag_name_f = 'favorite{}'.format(link_count) + callback_f = (lambda event, href=link: self.remove_favorite(event, href)) + self.site_display.tag_bind(tag_name_f, '', callback_f) + self.site_display.insert(tk.END, x['description'], (tag_name, tag_name_f, 'linkcolor')) + else: + self.site_display.insert(tk.END, x['description'], (tag_name,'linkcolor')) self.site_display.insert(tk.END, '\n') link_count += 1 self.site_display.config(state=tk.DISABLED) - return True + return link_count def show_text(self, data): @@ -380,12 +392,13 @@ class GUI: self.site_display.config(state=tk.DISABLED) - def send_to_screen(self, data, itemtype='1', clear=True): + def send_to_screen(self, data, itemtype='1', clear=True, link_count=0): if itemtype == '0': self.show_text(data) elif itemtype in ['1','3','7']: data = self.parser.parse_menu(data) - self.show_menu(data, clear) + links = self.show_menu(data, clear, link_count) + return links elif itemtype in ['p','I','g']: self.show_image(data) @@ -458,6 +471,17 @@ class GUI: self.root.destroy() + def remove_favorite(self, event, href): + print('remove!') + index = None + for ind, val in enumerate(self.config['favorites']): + if val['url'] == href: + index = ind + break + if index is not None: + self.config['favorites'].pop(index) + self.load_home_screen() + if __name__ == '__main__': app = GUI() From daa33307602c404dc48c855214902906ebe86b65 Mon Sep 17 00:00:00 2001 From: sloumdrone Date: Thu, 8 Nov 2018 20:25:27 -0800 Subject: [PATCH 5/5] Added context menu to delete favorites --- gui.py | 36 +++++++++++++++++++++++------------- 1 file changed, 23 insertions(+), 13 deletions(-) diff --git a/gui.py b/gui.py index ecbfc42..ec0d4a7 100644 --- a/gui.py +++ b/gui.py @@ -17,7 +17,7 @@ class GUI: self.history = [] self.history_location = -1 self.message_bar_content = '' - self.config = None + # self.config = None self.read_config() self.conn = conn() self.parser = parser() @@ -76,6 +76,9 @@ class GUI: #status bar objects self.status_info = tk.Label(self.status_bar, textvariable=self.message_bar_content, bg=self.STATUS_BG, takefocus=0, fg=self.ACTIVELINK) + #menu objects + self.fave_context = tk.Menu(self.body, tearoff=0) + self.pack_geometry() self.add_status_titles() @@ -145,6 +148,14 @@ class GUI: self.message_bar_content.set('Ready.') + def show_fave_context(self, e, href): + if self.fave_context.type(0): + self.fave_context.delete(0) + delete_favorite = (lambda event=e, link=href: self.remove_favorite(event, link)) + self.fave_context.add_command(label="Remove favorite", command=delete_favorite) + self.fave_context.tk_popup(e.x_root, e.y_root) + + # ------------Start navigation methods---------------------------- def handle_request(self,event=False, url=False, history=True): @@ -233,7 +244,7 @@ class GUI: self.add_to_history('home') data2 = self.load_favorites() link_count = self.send_to_screen(data, '1', True) - self.send_to_screen(data2, '1', False, link_count) + self.send_to_screen(data2, '1', False) def go_back(self, event): @@ -308,7 +319,7 @@ class GUI: self.search = None - def show_menu(self, data, clear=True, links=0): + def show_menu(self, data, clear=True): if not data: #error handling will go here return False @@ -332,7 +343,8 @@ class GUI: if clear: self.site_display.delete(1.0, tk.END) - link_count = links + if clear: + self.link_count = 0 for x in data: if x['type'] == 'i': @@ -351,7 +363,7 @@ class GUI: else: link = 'gopher://{}{}/{}{}'.format(x['host'], x['port'], x['type'], x['resource']) - tag_name = 'link{}'.format(link_count) + tag_name = 'link{}'.format(self.link_count) callback = (lambda event, href=link, tag_name=tag_name: self.gotolink(event, href, tag_name)) hover = (lambda event, href=link, tag_name=tag_name: self.hoverlink(event, href, tag_name)) favorite = [x for x in self.config['favorites'] if x['url'] == link] @@ -362,17 +374,17 @@ class GUI: self.site_display.insert(tk.END, types[x['type']], ('type_tag',)) self.site_display.insert(tk.END,'\t\t') if favorite: - tag_name_f = 'favorite{}'.format(link_count) - callback_f = (lambda event, href=link: self.remove_favorite(event, href)) + tag_name_f = 'favorite{}'.format(self.link_count) + callback_f = (lambda event, href=link: self.show_fave_context(event, href)) self.site_display.tag_bind(tag_name_f, '', callback_f) self.site_display.insert(tk.END, x['description'], (tag_name, tag_name_f, 'linkcolor')) else: self.site_display.insert(tk.END, x['description'], (tag_name,'linkcolor')) self.site_display.insert(tk.END, '\n') - link_count += 1 + self.link_count += 1 self.site_display.config(state=tk.DISABLED) - return link_count + return self.link_count def show_text(self, data): @@ -392,13 +404,12 @@ class GUI: self.site_display.config(state=tk.DISABLED) - def send_to_screen(self, data, itemtype='1', clear=True, link_count=0): + def send_to_screen(self, data, itemtype='1', clear=True): if itemtype == '0': self.show_text(data) elif itemtype in ['1','3','7']: data = self.parser.parse_menu(data) - links = self.show_menu(data, clear, link_count) - return links + self.show_menu(data, clear) elif itemtype in ['p','I','g']: self.show_image(data) @@ -472,7 +483,6 @@ class GUI: def remove_favorite(self, event, href): - print('remove!') index = None for ind, val in enumerate(self.config['favorites']): if val['url'] == href: