From 34dfa53b9f4f05a8afd051eda55dc4c79371b6dc Mon Sep 17 00:00:00 2001 From: Shawn Hind Date: Thu, 12 Mar 2015 17:42:26 -0400 Subject: [PATCH] Fixed suggested changes for pull request, changed colour of flairs to yellow for more visibility on submission page --- rtv/content.py | 10 ++-------- rtv/submission.py | 8 +++++--- rtv/subreddit.py | 5 ++--- 3 files changed, 9 insertions(+), 14 deletions(-) diff --git a/rtv/content.py b/rtv/content.py index 44d1356..bcbb58f 100644 --- a/rtv/content.py +++ b/rtv/content.py @@ -141,10 +141,7 @@ class BaseContent(object): sub_author = (clean(comment.submission.author.name) if getattr(comment.submission, 'author') else '[deleted]') data['is_author'] = (data['author'] == sub_author) - if comment.author_flair_text != None: - data['flair'] = comment.author_flair_text - else: - data['flair'] = "" + data['flair'] = (clean(comment.author_flair_text) if comment.author_flair_text else "") return data @@ -170,10 +167,7 @@ class BaseContent(object): else '[deleted]') data['permalink'] = clean(sub.permalink) data['subreddit'] = strip_subreddit_url(sub.permalink) - if sub.link_flair_text != None: - data['flair'] = sub.link_flair_text - else: - data['flair'] = "" + data['flair'] = (clean(sub.link_flair_text) if sub.link_flair_text else "") data['url_full'] = clean(sub.url) data['url'] = ('selfpost' if is_selfpost(sub.url) else clean(sub.url)) diff --git a/rtv/submission.py b/rtv/submission.py index 67c33e5..f56766a 100644 --- a/rtv/submission.py +++ b/rtv/submission.py @@ -107,10 +107,12 @@ class SubmissionPage(BasePage): row = offset if row in valid_rows: - text = '{author} {flair}'.format(**data) + text = '{author}'.format(**data) attr = curses.A_BOLD attr |= (Color.BLUE if not data['is_author'] else Color.GREEN) win.addnstr(row, 1, text, n_cols-1, attr) + text = ' {flair}'.format(**data) + win.addnstr(text, n_cols-win.getyx()[1], curses.A_BOLD | Color.YELLOW) text = ' {score} {created}'.format(**data) win.addnstr(text, n_cols - win.getyx()[1]) @@ -171,8 +173,8 @@ class SubmissionPage(BasePage): attr = curses.A_BOLD | Color.GREEN text = '{author}'.format(**data) win.addnstr(row, 1, text, n_cols, attr) - text = '{flair}'.format(**data) - win.addnstr(row, 1, text, n_cols, curses.A_BOLD | Color.GREEN) + text = ' {flair}'.format(**data) + win.addnstr(text, n_cols-win.getyx()[1], curses.A_BOLD | Color.YELLOW) text = ' {created} {subreddit}'.format(**data) win.addnstr(text, n_cols - win.getyx()[1]) diff --git a/rtv/subreddit.py b/rtv/subreddit.py index 1fa18f7..989a727 100644 --- a/rtv/subreddit.py +++ b/rtv/subreddit.py @@ -135,8 +135,7 @@ class SubredditPage(BasePage): if row in valid_rows: text = '{author}'.format(**data) win.addnstr(row, 1, text, n_cols-1, curses.A_BOLD) - if data['flair'] != None: - text = ' {flair}'.format(**data) - win.addnstr(text, n_cols - 1, curses.A_BOLD | Color.GREEN) + text = ' {flair}'.format(**data) + win.addnstr(text, n_cols - 1, curses.A_BOLD | Color.GREEN) text = ' {subreddit}'.format(**data) win.addnstr(text, n_cols - win.getyx()[1], Color.YELLOW)