Project

General

Profile

Actions

Patch #37507

closed

Normalize HTML in app/views/settings/_users.html.erb

Added by Holger Just over 1 year ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

In other settings pages, if there are tabular settings, they are encased in a <div class="tabular settings"> rather than adding the classes fdirectly to the fieldset (e.g. in _authorization.html.erb). This patch adjusts the generated HTML accordingly.

It also removes some special CSS rules for these setting pages which were introduced in r13600. I'm actually not really sure why these rules are there. In a default Redmine, they only move things slightly around in an inconsistent way. With custom themes, these rules have the possibility to mess things up.

In any case, I think those rules can just be removed outright:

  • splitcontentleft / splitcontentright is not used on this partial at all
  • the fieldset and labels should be rendered the same way as on all other pages instead of slightly different paddings / margins.
  • there is no tag with a principal id there

Files

Actions

Also available in: Atom PDF