tailwind_merge 0.15.0 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d9e79ce87f36904f2dd66beecff8d32e83b97025d729f6c324f14da53aca7c8e
-  data.tar.gz: 6ca2a49f5f9039028a9f6677bca5ec2f8765c52887680dd41c47449fe1770f28
+  metadata.gz: 3334600a104bc6b8efbe365c3be28e32944468f9cb5b1255fdefd4d106dd9540
+  data.tar.gz: 2f8c9a9b989c3e808767516bff6b040ed0c3a092a5fc427aaacaae4904523bcc
 SHA512:
-  metadata.gz: 20b3f51eb10c68b29b5f4354570ba6cdfbcab13103d7a45f213aecdaaaa50f5c9876b676f6a217c4b0a98335cffce007f1926f0f74b2c98af0155174b48c13d2
-  data.tar.gz: fe5786fa019198ca80233bceec9acdd5d9bd6d8098907143600d1df1e5a7a2b6525daa329c146c6d741c1df0dd276f7a87734f718cd865b7ad734d374bc6feed
+  metadata.gz: 7f8ad1a5e8dfc2386b81cf5f528e2b1b5ded8b9644dd942e777e3b356b0807d74397400c090b52aec68d54e93bc0bd8d672f9872795dd300b7e2b06b543701af
+  data.tar.gz: 33cc99d39060f7f2e6827d48f99bce9989df5771f0174eea1994133a25e05115855de5e4f138b763f7197801595c3ab8e13802291675786f782f7246429079dd

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+# [v0.16.0] - 25-01-2025
+## What's Changed
+* Fix and improve implementation by @w-masahiro-ct in https://github.com/gjtorikian/tailwind_merge/pull/50
+
+
+**Full Changelog**: https://github.com/gjtorikian/tailwind_merge/compare/v0.15.0...v0.16.0
 # [v0.15.0] - 23-01-2025
 ## What's Changed
 * Improve gemspec by @w-masahiro-ct in https://github.com/gjtorikian/tailwind_merge/pull/41

data/lib/tailwind_merge/class_utils.rb

@@ -16,7 +16,7 @@ module TailwindMerge
     def class_group_id(class_name)
       class_parts = class_name.split(CLASS_PART_SEPARATOR)
 
-      # Classes like `-inset-1` produce an empty string as first classPart.
+      # Classes like `-inset-1` produce an empty string as first class_part.
       # Assume that classes for negative values are used correctly and remove it from class_parts.
       class_parts.shift if class_parts.first == "" && class_parts.length != 1
 
@@ -30,9 +30,10 @@ module TailwindMerge
 
       next_class_part_object = class_part_object[:next_part][current_class_part]
 
-      class_group_from_next_class_part = next_class_part_object ? get_group_recursive(class_parts[1..-1], next_class_part_object) : nil
-
-      return class_group_from_next_class_part if class_group_from_next_class_part
+      if next_class_part_object
+        class_group_from_next_class_part = get_group_recursive(class_parts.drop(1), next_class_part_object)
+        return class_group_from_next_class_part if class_group_from_next_class_part
+      end
 
       return if class_part_object[:validators].empty?
 
@@ -40,29 +41,23 @@ module TailwindMerge
 
       result = class_part_object[:validators].find do |v|
         validator = v[:validator]
-
-        if from_theme?(validator)
-          validator.call(@config)
-        else
-          validator.call(class_rest)
-        end
+        from_theme?(validator) ? validator.call(@config) : validator.call(class_rest)
       end
 
-      result.nil? ? result : result[:class_group_id]
+      result&.fetch(:class_group_id, nil)
     end
 
     def get_conflicting_class_group_ids(class_group_id, has_postfix_modifier)
       conflicts = @config[:conflicting_class_groups][class_group_id] || []
 
       if has_postfix_modifier && @config[:conflicting_class_group_modifiers][class_group_id]
-        return [...conflicts, ...@config[:conflicting_class_group_modifiers][class_group_id]]
+        return [*conflicts, *@config[:conflicting_class_group_modifiers][class_group_id]]
       end
 
       conflicts
     end
 
     private def create_class_map(config)
-      theme = config[:theme]
       prefix = config[:prefix]
       class_map = {
         next_part: {},
@@ -70,12 +65,12 @@ module TailwindMerge
       }
 
       prefixed_class_group_entries = get_prefixed_class_group_entries(
-        config[:class_groups].map { |cg| [cg[0], cg[1]] },
+        config[:class_groups].map { |group_id, group_classes| [group_id, group_classes] },
         prefix,
       )
 
-      prefixed_class_group_entries.each do |(class_group_id, class_group)|
-        process_classes_recursively(class_group, class_map, class_group_id, theme)
+      prefixed_class_group_entries.each do |class_group_id, class_group|
+        process_classes_recursively(class_group, class_map, class_group_id)
       end
 
       class_map
@@ -84,53 +79,43 @@ module TailwindMerge
     private def get_prefixed_class_group_entries(class_group_entries, prefix)
       return class_group_entries if prefix.nil?
 
-      class_group_entries.map do |(class_group_id, class_group)|
+      class_group_entries.map do |class_group_id, class_group|
         prefixed_class_group = class_group.map do |class_definition|
-          next("#{prefix}#{class_definition}") if class_definition.is_a?(String)
-
-          next(class_definition.transform_keys { |key| "#{prefix}#{key}" }) if class_definition.is_a?(Hash)
-
-          class_definition
+          if class_definition.is_a?(String)
+            "#{prefix}#{class_definition}"
+          elsif class_definition.is_a?(Hash)
+            class_definition.transform_keys { |key| "#{prefix}#{key}" }
+          else
+            class_definition
+          end
         end
 
         [class_group_id, prefixed_class_group]
       end
     end
 
-    private def process_classes_recursively(class_group, class_part_object, class_group_id, theme)
+    private def process_classes_recursively(class_group, class_part_object, class_group_id)
       class_group.each do |class_definition|
         if class_definition.is_a?(String)
           class_part_object_to_edit = class_definition.empty? ? class_part_object : get_class_part(class_part_object, class_definition)
           class_part_object_to_edit[:class_group_id] = class_group_id
-          next
-        end
-
-        if class_definition.is_a?(Proc)
+        elsif class_definition.is_a?(Proc)
           if from_theme?(class_definition)
+            process_classes_recursively(class_definition.call(@config), class_part_object, class_group_id)
+          else
+            class_part_object[:validators] << {
+              validator: class_definition,
+              class_group_id: class_group_id,
+            }
+          end
+        else
+          class_definition.each do |key, nested_class_group|
             process_classes_recursively(
-              class_definition.call(@config),
-              class_part_object,
+              nested_class_group,
+              get_class_part(class_part_object, key),
               class_group_id,
-              theme,
             )
-            next
           end
-
-          class_part_object[:validators].push({
-            validator: class_definition,
-            class_group_id: class_group_id,
-          })
-
-          next
-        end
-
-        class_definition.each do |(key, class_group)|
-          process_classes_recursively(
-            class_group,
-            get_class_part(class_part_object, key),
-            class_group_id,
-            theme,
-          )
         end
       end
     end
@@ -153,16 +138,13 @@ module TailwindMerge
     end
 
     private def get_group_id_for_arbitrary_property(class_name)
-      if ARBITRARY_PROPERTY_REGEX.match?(class_name)
-        match = ARBITRARY_PROPERTY_REGEX.match(class_name) || ""
-        arbitrary_property_class_name = match[1] || ""
-        property = arbitrary_property_class_name[0...arbitrary_property_class_name.index(":")]
-
-        if !property.nil? && !property.empty?
-          # uses two dots here because one dot is used as prefix for class groups in plugins
-          "arbitrary..#{property}"
-        end
-      end
+      match = ARBITRARY_PROPERTY_REGEX.match(class_name)
+      return unless match
+
+      property = match[1].to_s.split(":", 2).first
+
+      # Use two dots here because one dot is used as prefix for class groups in plugins
+      "arbitrary..#{property}" if property && !property.empty?
     end
 
     private def from_theme?(validator)

data/lib/tailwind_merge/config.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "set"
+
 module TailwindMerge
   module Config
     include Validators
@@ -30,7 +32,7 @@ module TailwindMerge
       "skew",
       "space",
       "translate",
-    ]
+    ].freeze
     THEME_KEYS.each do |key|
       const_set(key.upcase.tr("-", "_"), ->(config) { config[:theme].fetch(key, nil) })
     end

data/lib/tailwind_merge/modifier_utils.rb

@@ -4,21 +4,20 @@ module TailwindMerge
   module ModifierUtils
     IMPORTANT_MODIFIER = "!"
 
-    def split_modifiers(class_name, separator: nil)
-      separator ||= ":"
+    def split_modifiers(class_name, separator: ":")
       separator_length = separator.length
-      seperator_is_single_char = separator_length == 1
-      first_seperator_char = separator[0]
+      separator_is_single_char = (separator_length == 1)
+      first_separator_char = separator[0]
 
       modifiers = []
       bracket_depth = 0
       modifier_start = 0
-      postfix_modifier_position = 0
+      postfix_modifier_position = nil
 
       class_name.each_char.with_index do |char, index|
         if bracket_depth.zero?
-          if char == first_seperator_char && (seperator_is_single_char || class_name[index..(index + separator_length - 1)] == separator)
-            modifiers << class_name[modifier_start..index]
+          if char == first_separator_char && (separator_is_single_char || class_name[index, separator_length] == separator)
+            modifiers << class_name[modifier_start...index]
             modifier_start = index + separator_length
             next
           elsif char == "/"
@@ -27,17 +26,17 @@ module TailwindMerge
           end
         end
 
-        if char == "["
-          bracket_depth += 1
-        elsif char == "]"
-          bracket_depth -= 1
-        end
+        bracket_depth += 1 if char == "["
+        bracket_depth -= 1 if char == "]"
       end
 
-      base_class_name_with_important_modifier = modifiers.empty? ? class_name : class_name[modifier_start..-1]
+      base_class_name_with_important_modifier = modifiers.empty? ? class_name : class_name[modifier_start..]
       has_important_modifier = base_class_name_with_important_modifier.start_with?(IMPORTANT_MODIFIER)
-      base_class_name = has_important_modifier ? base_class_name_with_important_modifier[1..-1] : base_class_name_with_important_modifier
-      maybe_postfix_modifier_position = postfix_modifier_position && postfix_modifier_position > modifier_start ? postfix_modifier_position - modifier_start : false
+      base_class_name = has_important_modifier ? base_class_name_with_important_modifier[1..] : base_class_name_with_important_modifier
+
+      maybe_postfix_modifier_position = if postfix_modifier_position && postfix_modifier_position > modifier_start
+        postfix_modifier_position - modifier_start
+      end
 
       [modifiers, has_important_modifier, base_class_name, maybe_postfix_modifier_position]
     end
@@ -46,25 +45,22 @@ module TailwindMerge
     # - Predefined modifiers are sorted alphabetically
     # - When an arbitrary variant appears, it must be preserved which modifiers are before and after it
     def sort_modifiers(modifiers)
-      if modifiers.length <= 1
-        return modifiers
-      end
+      return modifiers if modifiers.size <= 1
 
       sorted_modifiers = []
       unsorted_modifiers = []
 
       modifiers.each do |modifier|
-        is_arbitrary_variant = modifier[0] == "["
-
-        if is_arbitrary_variant
-          sorted_modifiers.push(unsorted_modifiers.sort, modifier)
-          unsorted_modifiers = []
+        if modifier.start_with?("[")
+          sorted_modifiers.concat(unsorted_modifiers.sort)
+          sorted_modifiers << modifier
+          unsorted_modifiers.clear
         else
-          unsorted_modifiers.push(modifier)
+          unsorted_modifiers << modifier
         end
       end
 
-      sorted_modifiers.push(...unsorted_modifiers.sort)
+      sorted_modifiers.concat(unsorted_modifiers.sort)
 
       sorted_modifiers
     end

data/lib/tailwind_merge/validators.rb

@@ -8,12 +8,10 @@ module TailwindMerge
       def arbitrary_value?(class_part, label, test_value)
         match = ARBITRARY_VALUE_REGEX.match(class_part)
         return false unless match
+        return test_value.call(match[2]) if match[1].nil?
+        return label == match[1] if label.is_a?(String)
 
-        unless match[1].nil?
-          return label.is_a?(Set) ? label.include?(match[1]) : label == match[1]
-        end
-
-        test_value.call(match[2])
+        label.include?(match[1])
       end
 
       def numeric?(x)

data/lib/tailwind_merge/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TailwindMerge
-  VERSION = "0.15.0"
+  VERSION = "0.16.0"
 end

data/lib/tailwind_merge.rb

@@ -19,7 +19,7 @@ module TailwindMerge
     SPLIT_CLASSES_REGEX = /\s+/
 
     def initialize(config: {})
-      @config = if config.fetch(:theme, nil)
+      @config = if config.key?(:theme)
         merge_configs(config)
       else
         TailwindMerge::Config::DEFAULTS.merge(config)
@@ -30,12 +30,10 @@ module TailwindMerge
     end
 
     def merge(classes)
-      if classes.is_a?(Array)
-        classes = classes.compact.join(" ")
-      end
+      normalized = classes.is_a?(Array) ? classes.compact.join(" ") : classes.to_s
 
-      @cache.getset(classes) do
-        merge_class_list(classes).freeze
+      @cache.getset(normalized) do
+        merge_class_list(normalized).freeze
       end
     end
 
@@ -45,37 +43,38 @@ module TailwindMerge
       # @example 'float'
       # @example 'hover:focus:bg-color'
       # @example 'md:!pr'
-      class_groups_in_conflict = []
-      class_names = class_list.strip.split(SPLIT_CLASSES_REGEX)
-
-      result = ""
+      trimmed = class_list.strip
+      return "" if trimmed.empty?
 
-      i = class_names.length - 1
+      class_groups_in_conflict = Set.new
 
-      loop do
-        break if i < 0
+      merged_classes = []
 
-        original_class_name = class_names[i]
+      trimmed.split(SPLIT_CLASSES_REGEX).reverse_each do |original_class_name|
+        modifiers, has_important_modifier, base_class_name, maybe_postfix_modifier_position =
+          split_modifiers(original_class_name, separator: @config[:separator])
 
-        modifiers, has_important_modifier, base_class_name, maybe_postfix_modifier_position = split_modifiers(original_class_name, separator: @config[:separator])
+        actual_base_class_name = if maybe_postfix_modifier_position
+          base_class_name[0...maybe_postfix_modifier_position]
+        else
+          base_class_name
+        end
 
-        actual_base_class_name = maybe_postfix_modifier_position ? base_class_name[0...maybe_postfix_modifier_position] : base_class_name
+        has_postfix_modifier = maybe_postfix_modifier_position ? true : false
         class_group_id = @class_utils.class_group_id(actual_base_class_name)
 
         unless class_group_id
-          unless maybe_postfix_modifier_position
-            # not a Tailwind class
-            result = original_class_name + (!result.empty? ? " " + result : result)
-            i -= 1
+          unless has_postfix_modifier
+            # Not a Tailwind class
+            merged_classes << original_class_name
             next
           end
 
           class_group_id = @class_utils.class_group_id(base_class_name)
 
           unless class_group_id
-            # not a Tailwind class
-            result = original_class_name + (!result.empty? ? " " + result : result)
-            i -= 1
+            # Not a Tailwind class
+            merged_classes << original_class_name
             next
           end
 
@@ -87,25 +86,20 @@ module TailwindMerge
         modifier_id = has_important_modifier ? "#{variant_modifier}#{IMPORTANT_MODIFIER}" : variant_modifier
         class_id = "#{modifier_id}#{class_group_id}"
 
-        # Tailwind class omitted due to pre-existing conflict
-        if class_groups_in_conflict.include?(class_id)
-          i -= 1
-          next
-        end
+        # Tailwind class omitted due to conflict
+        next if class_groups_in_conflict.include?(class_id)
 
-        class_groups_in_conflict.push(class_id)
+        class_groups_in_conflict << class_id
 
-        @class_utils.get_conflicting_class_group_ids(class_group_id, has_postfix_modifier).each do |group|
-          class_groups_in_conflict.push("#{modifier_id}#{group}")
+        @class_utils.get_conflicting_class_group_ids(class_group_id, has_postfix_modifier).each do |conflicting_id|
+          class_groups_in_conflict << "#{modifier_id}#{conflicting_id}"
         end
 
-        # no conflict!
-        result = original_class_name + (!result.empty? ? " " + result : result)
-
-        i -= 1
+        # Tailwind class not in conflict
+        merged_classes << original_class_name
       end
 
-      result
+      merged_classes.reverse.join(" ")
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tailwind_merge
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.16.0
 platform: ruby
 authors:
 - Garen J. Torikian
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-01-23 00:00:00.000000000 Z
+date: 2025-01-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sin_lru_redux
@@ -74,15 +74,15 @@ files:
 - lib/tailwind_merge/validators.rb
 - lib/tailwind_merge/version.rb
 - tailwind_merge.gemspec
-homepage: https://github.com/gjtorikian/tailwind_merge/tree/v0.15.0
+homepage: https://github.com/gjtorikian/tailwind_merge/tree/v0.16.0
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/gjtorikian/tailwind_merge/tree/v0.15.0
-  source_code_uri: https://github.com/gjtorikian/tailwind_merge/tree/v0.15.0
-  changelog_uri: https://github.com/gjtorikian/tailwind_merge/blob/v0.15.0/CHANGELOG.md
+  homepage_uri: https://github.com/gjtorikian/tailwind_merge/tree/v0.16.0
+  source_code_uri: https://github.com/gjtorikian/tailwind_merge/tree/v0.16.0
+  changelog_uri: https://github.com/gjtorikian/tailwind_merge/blob/v0.16.0/CHANGELOG.md
   bug_tracker_uri: https://github.com/gjtorikian/tailwind_merge/issues
-  documentation_uri: https://rubydoc.info/gems/tailwind_merge/0.15.0
+  documentation_uri: https://rubydoc.info/gems/tailwind_merge/0.16.0
   funding_uri: https://github.com/sponsors/gjtorikian
   rubygems_mfa_required: 'true'
 post_install_message: