プロジェクト

全般

プロフィール

Vote #76930

完了

Wiki lists in the sidebar are broken

Admin Redmine さんが3年以上前に追加. 3年以上前に更新.

ステータス:
Closed
優先度:
通常
担当者:
-
カテゴリ:
UI_10
対象バージョン:
開始日:
2022/05/09
期日:
進捗率:

0%

予定工数:
category_id:
10
version_id:
112
issue_org_id:
21263
author_id:
80755
assigned_to_id:
1
comments:
8
status_id:
5
tracker_id:
1
plus1:
0
affected_version:
closed_on:
affected_version_id:
ステータス-->[Closed]

説明

The list in the Sidebar didn't work anymore since 2.5.X version. The same problem appear with

{{child_pages}}
macro.

I tested on demo.redmine.org with the same result.

But if you put an orderer list (with #) it works.

This is due to changes in the Redmine CSS in 2.4.0 that added "lines 80 and 81":/projects/redmine/repository/annotate/trunk/public/stylesheets/application.css#L80 in the main Redmine CSS, see also #13242.

This is my redmine:

Redmine 2.6
Ruby 2.2

journals

Minimal fix for the problem attached:
* Wrap the wiki text a div.wiki (which is required anyway for proper wiki text styling)
* Undo the styles of form @#sidebar xyz@ for @#sidebar div.wiki xyz@ which works but is a bit ugly

The proper fix (IMHO) would be to wrap all content of the sidebar (except the wiki) in its own div with a class, say "content" and then match everything with @#sidebar .content xyz@. This would leave the .wiki stuff unaffected and would not require dirty "counter"-rules in application.css like I did in the minimal fix here.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Daniel Ritz wrote:
> The proper fix (IMHO) would be to wrap all content of the sidebar (except the wiki) in its own div with a class, say "content" and then match everything with @#sidebar .content xyz@. This would leave the .wiki stuff unaffected and would not require dirty "counter"-rules in application.css like I did in the minimal fix here.

Should I implement that or is the minimal fix enough?

--------------------------------------------------------------------------------
Daniel Ritz wrote:

> The proper fix (IMHO) would be to wrap all content of the sidebar (except the wiki) in its own div with a class

Would you do that by adding a div in each view where there is some content for the sidebar?
If so, I'd prefer the patch you've already submitted.
--------------------------------------------------------------------------------
Jean-Philippe Lang wrote:
> Daniel Ritz wrote:
>
> > The proper fix (IMHO) would be to wrap all content of the sidebar (except the wiki) in its own div with a class
>
> Would you do that by adding a div in each view where there is some content for the sidebar?
> If so, I'd prefer the patch you've already submitted.

Yeah, pretty much. Thinking about it some more, the minimal fix is better as it allows sidebar content at any level to insert .wiki and formatted text that just works.
--------------------------------------------------------------------------------
Patch committed, thanks for the feedback.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------


related_issues

relates,Closed,13242,Use <li> tags for lists

Admin Redmine さんが3年以上前に更新

  • カテゴリUI_10 にセット
  • 対象バージョン3.2.1_112 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0