Skip to content

Commit ee2d321

Browse files
committed
Only move loading bar to top if there's a render to show
1 parent 263036c commit ee2d321

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

app/components/renderer.rb

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ def render?
1414

1515
def before_template
1616
@settings = current_user&.renderer_settings || SiteSettings::UserDefaults::RENDERER
17+
@loading_bar_position_classes = @file.has_render? ?
18+
"btn-sm mt-3 translate-middle-x translate-top" :
19+
"top-50 translate-middle"
1720
end
1821

1922
def view_template
@@ -37,7 +40,7 @@ def view_template
3740
render_style: @settings["render_style"],
3841
auto_load: ((@file.size || 9_999_999.megabytes) < (@settings["auto_load_max_size"] || 9_999_999).megabytes) ? "true" : "false"
3942
}
40-
div class: "p-0 btn btn-sm btn-secondary load-progress object-preview-progress position-absolute mt-3 start-50 translate-middle-x translate-top", role: "presentation" do
43+
div class: "p-0 btn btn-secondary load-progress object-preview-progress position-absolute start-50 #{@loading_bar_position_classes}", role: "presentation" do
4144
div class: "progress-bar bg-info progress-bar-animated progress-bar-striped", role: "progressbar", style: "width: 0%; height: 100%",
4245
aria_label: "Loading progress", aria_valuenow: "0", aria_valuemin: "0", aria_valuemax: "100"
4346
span class: "progress-label position-absolute top-50 start-50 translate-middle", role: "button" do

0 commit comments

Comments
 (0)