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 about 1 year ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Category:
UI - Responsive
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
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

Actions #1

Updated by Jens Krämer about 1 year 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 about 1 year ago

  • Target version set to 6.0.0
Actions #3

Updated by Marius BĂLTEANU 12 months 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 12 months 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 12 months ago

  • Status changed from Resolved to Closed
Actions #6

Updated by Mizuki ISHIKAWA 11 months 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 11 months ago

I'll look into this

Actions #8

Updated by Jens Krämer 11 months ago

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

Actions #9

Updated by Mizuki ISHIKAWA 11 months 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 11 months ago

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

Actions #12

Updated by Mizuki ISHIKAWA 11 months ago

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

Actions #13

Updated by Marius BĂLTEANU 11 months ago

Mizuki ISHIKAWA wrote in #note-12:

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

Thanks, committed.

Actions

Also available in: Atom PDF