diff --git a/frontend/ui/widget/textviewer.lua b/frontend/ui/widget/textviewer.lua index 4a7f888f8..d1828e5eb 100644 --- a/frontend/ui/widget/textviewer.lua +++ b/frontend/ui/widget/textviewer.lua @@ -12,19 +12,16 @@ local BD = require("ui/bidi") local Blitbuffer = require("ffi/blitbuffer") local ButtonTable = require("ui/widget/buttontable") local CenterContainer = require("ui/widget/container/centercontainer") -local CloseButton = require("ui/widget/closebutton") local Device = require("device") local Geom = require("ui/geometry") local Font = require("ui/font") local FrameContainer = require("ui/widget/container/framecontainer") local GestureRange = require("ui/gesturerange") local InputContainer = require("ui/widget/container/inputcontainer") -local LineWidget = require("ui/widget/linewidget") local MovableContainer = require("ui/widget/container/movablecontainer") -local OverlapGroup = require("ui/widget/overlapgroup") local ScrollTextWidget = require("ui/widget/scrolltextwidget") local Size = require("ui/size") -local TextBoxWidget = require("ui/widget/textboxwidget") +local TitleBar = require("ui/widget/titlebar") local UIManager = require("ui/uimanager") local VerticalGroup = require("ui/widget/verticalgroup") local WidgetContainer = require("ui/widget/container/widgetcontainer") @@ -51,11 +48,11 @@ local TextViewer = InputContainer:new{ auto_para_direction = true, alignment_strict = false, - title_face = Font:getFace("x_smalltfont"), + title_face = nil, -- use default from TitleBar + title_multilines = nil, -- see TitleBar for details + title_shrink_font_to_fit = nil, -- see TitleBar for details text_face = Font:getFace("x_smallinfofont"), fgcolor = Blitbuffer.COLOR_BLACK, - title_padding = Size.padding.default, - title_margin = Size.margin.title, text_padding = Size.padding.large, text_margin = Size.margin.small, button_padding = Size.padding.default, @@ -103,43 +100,20 @@ function TextViewer:init() } end - local closeb = CloseButton:new{ window = self, padding_top = Size.padding.tiny, } - local title_text = TextBoxWidget:new{ - text = self.title, - face = self.title_face, - bold = true, - width = self.width - 2*self.title_padding - 2*self.title_margin - closeb:getSize().w, - } - local titlew = FrameContainer:new{ - padding = self.title_padding, - -- TextBoxWidget has less text top & bottom padding than TextWidget - -- (for a reasonable line height with multi lines), but we - -- can get the same as TextWidget by simply adding Size.padding.small - padding_top = self.title_padding + Size.padding.small, - padding_bottom = self.title_padding + Size.padding.small, - margin = self.title_margin, - bordersize = 0, - title_text - } - titlew = OverlapGroup:new{ - dimen = { - w = self.width, - h = titlew:getSize().h - }, - titlew, - closeb, - } - - local separator = LineWidget:new{ - dimen = Geom:new{ - w = self.width, - h = Size.line.thick, - } + local titlebar = TitleBar:new{ + width = self.width, + align = "left", + with_bottom_line = true, + title = self.title, + title_face = self.title_face, + title_multilines = self.title_multilines, + title_shrink_font_to_fit = self.title_shrink_font_to_fit, + close_callback = function() self:onClose() end, + show_parent = self, } - local buttons - if self.buttons_table == nil then - buttons = { + local buttons = self.buttons_table or + { { { text = _("Close"), @@ -149,19 +123,14 @@ function TextViewer:init() }, }, } - else - buttons = self.buttons_table - end local button_table = ButtonTable:new{ width = self.width - 2*self.button_padding, - button_font_face = "cfont", - button_font_size = 20, buttons = buttons, zero_sep = true, show_parent = self, } - local textw_height = self.height - titlew:getSize().h - separator:getSize().h - button_table:getSize().h + local textw_height = self.height - titlebar:getHeight() - button_table:getSize().h self.scroll_text_w = ScrollTextWidget:new{ text = self.text, @@ -190,9 +159,7 @@ function TextViewer:init() margin = 0, background = Blitbuffer.COLOR_WHITE, VerticalGroup:new{ - align = "left", - titlew, - separator, + titlebar, CenterContainer:new{ dimen = Geom:new{ w = self.width,