forked from openlp/openlp
Fix some todo notes
This commit is contained in:
parent
f7532b7896
commit
b41b4a5918
@ -80,7 +80,6 @@ class UiStrings(object):
|
|||||||
self.Export = translate('OpenLP.Ui', 'Export')
|
self.Export = translate('OpenLP.Ui', 'Export')
|
||||||
self.File = translate('OpenLP.Ui', 'File')
|
self.File = translate('OpenLP.Ui', 'File')
|
||||||
self.FileNotFound = translate('OpenLP.Ui', 'File Not Found')
|
self.FileNotFound = translate('OpenLP.Ui', 'File Not Found')
|
||||||
# TODO: Passed nose - verify in real life example
|
|
||||||
self.FileNotFoundMessage = translate('OpenLP.Ui',
|
self.FileNotFoundMessage = translate('OpenLP.Ui',
|
||||||
'File {name} not found.\nPlease try selecting it individually.')
|
'File {name} not found.\nPlease try selecting it individually.')
|
||||||
self.FontSizePtUnit = translate('OpenLP.Ui', 'pt', 'Abbreviated font pointsize unit')
|
self.FontSizePtUnit = translate('OpenLP.Ui', 'pt', 'Abbreviated font pointsize unit')
|
||||||
|
@ -51,7 +51,6 @@ class FileDialog(QtWidgets.QFileDialog):
|
|||||||
file = parse.unquote(file)
|
file = parse.unquote(file)
|
||||||
if not os.path.exists(file):
|
if not os.path.exists(file):
|
||||||
log.error('File {text} not found.'.format(text=file))
|
log.error('File {text} not found.'.format(text=file))
|
||||||
# TODO: Should work - need to verify
|
|
||||||
QtWidgets.QMessageBox.information(parent, UiStrings().FileNotFound,
|
QtWidgets.QMessageBox.information(parent, UiStrings().FileNotFound,
|
||||||
UiStrings().FileNotFoundMessage.format(name=file))
|
UiStrings().FileNotFoundMessage.format(name=file))
|
||||||
continue
|
continue
|
||||||
|
@ -371,7 +371,6 @@ class Renderer(OpenLPMixin, RegistryMixin, RegistryProperties):
|
|||||||
self.web.resize(self.page_width, self.page_height)
|
self.web.resize(self.page_width, self.page_height)
|
||||||
self.web_frame = self.web.page().mainFrame()
|
self.web_frame = self.web.page().mainFrame()
|
||||||
# Adjust width and height to account for shadow. outline done in css.
|
# Adjust width and height to account for shadow. outline done in css.
|
||||||
# TODO: Tested at home
|
|
||||||
html = Template("""<!DOCTYPE html><html><head><script>
|
html = Template("""<!DOCTYPE html><html><head><script>
|
||||||
function show_text(newtext) {
|
function show_text(newtext) {
|
||||||
var main = document.getElementById('main');
|
var main = document.getElementById('main');
|
||||||
|
@ -513,7 +513,7 @@ class ThemeXML(object):
|
|||||||
theme_strings = []
|
theme_strings = []
|
||||||
for key in dir(self):
|
for key in dir(self):
|
||||||
if key[0:1] != '_':
|
if key[0:1] != '_':
|
||||||
# TODO: Tested at home
|
# TODO: Due to bound methods returned, I don't know how to write a proper test
|
||||||
theme_strings.append('{key:>30}: {value}'.format(key=key, value=getattr(self, key)))
|
theme_strings.append('{key:>30}: {value}'.format(key=key, value=getattr(self, key)))
|
||||||
return '\n'.join(theme_strings)
|
return '\n'.join(theme_strings)
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user