How to reproduce

* Configure (not too unusual) from-prefix: "Min.", to-prefix: "Max."
* On display formatter settings,
* Enable from&to prefix/suffix (for the above this is needed)
* Use combined value display

Result with combined value (when from=to):
"Min. 123"

Expected: Something that makes sense.

Background

Comment from _range_field_formatter_view_combined_value():

If both (FROM & TO) are enabled, show prefix from FROM and suffix from TO

Proposed resolution

Add combined prefix/suffix.

CommentFileSizeAuthor
#2 range-2565061-2-Combined-display-prefix-suffix-flawed.patch3.33 KBgeek-merlin

Comments

axel.rutz created an issue. See original summary.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Active » Needs review
StatusFileSize
new range-2565061-2-Combined-display-prefix-suffix-flawed.patch3.33 KB

Patch flying in that implements this.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Related issues: +#2500769: Allow point range input

Crosslinking #2500769: Allow point range input

Status: Needs review » Needs work

The last submitted patch, 2: range-2565061-2-Combined-display-prefix-suffix-flawed.patch, failed testing.

taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented

Hi @axel.rutz

Thanks for the issue. Your use case seems valid. Could you check your patch and make sure it applies ?

Thanks!

  • Taran2L committed 2aa6e4f on 
    Issue #2565061 by axel.rutz, Taran2L: Add combined value prefix/suffix
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Category: Bug report » Feature request
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.