Project

General

Profile

Actions

Feature #39806

closed

Improve filter rendering on narrow screens by replacing the layout tables with a flex layout

Added by Jens Krämer over 2 years ago. Updated 18 days ago.

Status:
Closed
Priority:
Normal
Category:
UI - Responsive
Target version:
Resolution:
Fixed

Description

The current query filters and options use HTML tables for the layout, which introduces issues with wide form elements on narrow screens.

The attached patch which was extracted from Planio replaces the layout tables with a flex layout. The look and feel in desktop browsers should stay unchanged, but on narrow screens like mobile phones filter rows that previously just exceeded the screen width will now wrap to a new row.


Files


Related issues

Related to Redmine - Defect #44170: Toggling between board and list in projects query do not work properlyClosedMarius BĂLTEANUActions
Actions #1

Updated by Jens Krämer over 2 years ago

here's the patch :)

Also: this change has the potential to break plugins which use the changed partials and rely on the old table markup in their stylesheets or Javascript.

Actions #2

Updated by Marius BĂLTEANU over 2 years ago

  • Target version set to 6.0.0
Actions #3

Updated by Marius BĂLTEANU over 2 years ago

  • Tracker changed from Patch to Feature
  • Subject changed from Improve filter rendering on narrow screens to Improve filter rendering on narrow screens by replacing the layout tables with a flex layout
  • Status changed from New to Resolved
  • Assignee set to Marius BĂLTEANU
  • Resolution set to Fixed

Committed, thanks!

Actions #4

Updated by Marius BĂLTEANU over 2 years ago

Jens Krämer wrote in #note-1:

Also: this change has the potential to break plugins which use the changed partials and rely on the old table markup in their stylesheets or Javascript.

Redmine 6.0.0 is a good moment to deliver this potential breaking change.

Actions #5

Updated by Marius BĂLTEANU over 2 years ago

  • Status changed from Resolved to Closed
Actions #6

Updated by Mizuki ISHIKAWA over 2 years ago

Thanks for improving the layout. However, there seems to be one problem. The layout of the UI for selecting whether or not to display a column of issues is broken due to this change.
Where the problem occurs:
  • Administration => Settings => Projects tab => Projects list defaults
  • Administration => Settings => Projects tab => Issues list defaults
  • My page => Add: Reported issues => Click setting icon
Expected Actual
Actions #7

Updated by Jens Krämer over 2 years ago

I'll look into this

Actions #8

Updated by Jens Krämer over 2 years ago

attached is a patch that fixes the layout in all three places. second patch for whitespace / indentation.

Actions #9

Updated by Mizuki ISHIKAWA over 2 years ago

Jens Krämer wrote in #note-8:

attached is a patch that fixes the layout in all three places. second patch for whitespace / indentation.

Thank you for fixing the problem so quickly! After applying the patch, I was able to verify that the problem was fixed on the three reported screens.
I found a problem with "Administration => Settings => Time tracking tab => Timelog list defaults" as well. Sorry I didn't report it first.

Actions #11

Updated by Marius BĂLTEANU over 2 years ago

All three patches committed, thank you for reporting and fixing those issues so quickly.

Actions #12

Updated by Mizuki ISHIKAWA over 2 years ago

Thank you for your commit. I found one more same problem and fixed it.

Actions #13

Updated by Marius BĂLTEANU over 2 years ago

Mizuki ISHIKAWA wrote in #note-12:

Thank you for your commit. I found one more same problem and fixed it.

Thanks, committed.

Actions #14

Updated by Takashi Kato 18 days ago

As a result of the changes in this revision, the column selection toggle does not work properly.
The attached patch fix this issue.

Actions #15

Updated by Marius BĂLTEANU 18 days ago

  • Related to Defect #44170: Toggling between board and list in projects query do not work properly added
Actions #16

Updated by Marius BĂLTEANU 18 days ago

Takashi Kato wrote in #note-14:

As a result of the changes in this revision, the column selection toggle does not work properly.
The attached patch fix this issue.

I've committed the fix in #44170.

Actions

Also available in: Atom PDF