Probably fix layout issues caused by floating point errors where an element is wrapped when it has space (e.g. 2 elements with a width 0.5 not fitting properly)

lib/cyberarm_engine/background_image.rb

@@ -1,10 +1,11 @@
 module CyberarmEngine
   class BackgroundImage
     include CyberarmEngine::Common
-    attr_accessor :x, :y, :z, :width, :height, :mode, :color
-    attr_reader :image
+    attr_accessor :x, :y, :z, :mode
+    attr_reader :image, :width, :height, :color
 
     def initialize(image_path: nil, x: 0, y: 0, z: 0, width: 0, height: 0, mode: :fill, color: Gosu::Color::WHITE)
+      @image_path = image_path
       @image = get_image(image_path) if image_path
 
       @x = x
@@ -16,12 +17,31 @@ module CyberarmEngine
       @mode = mode
 
       @color = color
+
+      @cached_tiling = nil
     end
 
     def image=(image_path)
+      @cached_tiling = nil if image_path != @image_path
+      @image_path = image_path
       @image = image_path ? get_image(image_path) : image_path
     end
 
+    def width=(n)
+      @cached_tiling = nil if @width != n
+      @width = n
+    end
+
+    def height=(n)
+      @cached_tiling = nil if @height != n
+      @height = n
+    end
+
+    def color=(c)
+      @cached_tiling = nil if @color != c
+      @color = c
+    end
+
     def width_scale
       (@width.to_f / @image.width).abs
     end
@@ -33,7 +53,9 @@ module CyberarmEngine
     def draw
       return unless @image
 
-      send(:"draw_#{mode}")
+      Gosu.clip_to(@x, @y, @width, @height) do
+        send(:"draw_#{mode}")
+      end
     end
 
     def draw_stretch
@@ -41,15 +63,31 @@ module CyberarmEngine
     end
 
     def draw_tiled
-      raise NotImplementedError
+      @cached_tiling ||= Gosu.record(@width, @height) do
+        height_scale.ceil.times do |y|
+          width_scale.ceil.times do |x|
+            @image.draw(x * @image.width, y * @image.height, @z, 1, 1, @color)
+          end
+        end
+      end
+
+      @cached_tiling.draw(@x, @y, @z)
     end
 
     def draw_fill
       if @width * width_scale > height * height_scale
-        @image.draw(@x, @y, @z, width_scale, width_scale, @color)
+        draw_fill_width
       else
-        @image.draw(@x, @y, @z, height_scale, height_scale, @color)
+        draw_fill_height
       end
     end
+
+    def draw_fill_width
+      @image.draw(@x, @y, @z, width_scale, width_scale, @color)
+    end
+
+    def draw_fill_height
+      @image.draw(@x, @y, @z, height_scale, height_scale, @color)
+    end
   end
 end

lib/cyberarm_engine/ui/element.rb

@@ -441,7 +441,7 @@ module CyberarmEngine
       raise "dimension must be either :width or :height" unless %i[width height].include?(dimension)
 
       new_size = if size.is_a?(Numeric) && size.between?(0.0, 1.0)
-        (@parent.send(:"content_#{dimension}") * size).round - send(:"noncontent_#{dimension}").round
+        (@parent.send(:"content_#{dimension}") * size).floor - send(:"noncontent_#{dimension}").floor
       else
         size
       end
@@ -450,11 +450,15 @@ module CyberarmEngine
         fill_siblings = @parent.children.select { |c| c.style.fill }.count.to_f # include self since we're dividing
 
         if dimension == :width && @parent.is_a?(Flow)
-          space_available_width = ((@parent.content_width - (@parent.children.reject { |c| c.style.fill }).map(&:outer_width).sum) / fill_siblings).round
+          space_available_width = ((@parent.content_width - (@parent.children.reject { |c| c.style.fill }).map(&:outer_width).sum) / fill_siblings)
+          space_available_width = space_available_width.nan? ? 0 : space_available_width.floor # The parent element might not have its dimensions, yet.
+
           return space_available_width - noncontent_width
 
         elsif dimension == :height && @parent.is_a?(Stack)
-          space_available_height = ((@parent.content_height - (@parent.children.reject { |c| c.style.fill }).map(&:outer_height).sum) / fill_siblings).round
+          space_available_height = ((@parent.content_height - (@parent.children.reject { |c| c.style.fill }).map(&:outer_height).sum) / fill_siblings)
+          space_available_height = space_available_height.nan? ? 0 : space_available_height.floor # The parent element might not have its dimensions, yet.
+
           return space_available_height - noncontent_height
         end
 
@@ -509,8 +513,6 @@ module CyberarmEngine
     end
 
     def background_image=(image_path)
-      pp @style.background_image_canvas.image
-
       @style.background_image = image_path.is_a?(Gosu::Image) ? image_path : get_image(image_path)
       update_background_image
     end

lib/cyberarm_engine/ui/elements/button.rb

@@ -64,8 +64,8 @@ module CyberarmEngine
             @scale_y = 1
           end
 
-          @width = _width || @image.width.round * @scale_x
-          @height = _height || @image.height.round * @scale_y
+          @width = _width || @image.width.floor * @scale_x
+          @height = _height || @image.height.floor * @scale_y
 
           update_background
         else

lib/cyberarm_engine/ui/elements/container.rb

@@ -122,8 +122,8 @@ module CyberarmEngine
           _width = dimensional_size(@style.width, :width)
           _height = dimensional_size(@style.height, :height)
 
-          @width  = _width  || (@children.map { |c| c.x + c.outer_width }.max || 0).round
-          @height = _height || (@children.map { |c| c.y + c.outer_height }.max || 0).round
+          @width  = _width  || (@children.map { |c| c.x + c.outer_width }.max || 0).floor
+          @height = _height || (@children.map { |c| c.y + c.outer_height }.max || 0).floor
         end
 
         # Move child to parent after positioning

lib/cyberarm_engine/ui/elements/edit_box.rb

@@ -98,7 +98,7 @@ module CyberarmEngine
       end
 
       def row_at(y)
-        ((y - @text.y) / @text.textobject.height).round
+        ((y - @text.y) / @text.textobject.height).floor
       end
 
       def column_at(x, y)

lib/cyberarm_engine/ui/elements/image.rb

@@ -45,8 +45,8 @@ module CyberarmEngine
           @scale_y = 1
         end
 
-        @width = _width || @image.width.round * @scale_x
-        @height = _height || @image.height.round * @scale_y
+        @width = _width || @image.width.floor * @scale_x
+        @height = _height || @image.height.floor * @scale_y
 
         update_background
       end

lib/cyberarm_engine/ui/elements/list_box.rb

@@ -17,6 +17,8 @@ module CyberarmEngine
         @menu.define_singleton_method(:recalculate_menu) do
           @x = @__list_box.x
           @y = @__list_box.y + @__list_box.height
+
+          @y = @__list_box.y - height if @y + height > window.height
         end
         @menu.instance_variable_set(:"@__list_box", self)
 

lib/cyberarm_engine/ui/elements/text_block.rb

@@ -43,8 +43,8 @@ module CyberarmEngine
 
         handle_text_wrapping(_width)
 
-        @width  = _width  || @text.width.round
-        @height = _height || @text.height.round
+        @width  = _width  || @text.width.floor
+        @height = _height || @text.height.floor
 
         @text.y = @style.border_thickness_top + @style.padding_top + @y
         @text.z = @z + 3