Skip to content

Conversation

telegib
Copy link

@telegib telegib commented Jun 30, 2023

redmine.orgのチケットURL(feature): https://www.redmine.org/issues/38650
(patch のものはまだ未作成)

改善内容

  • トラッカーが多い場合、カスタムフィールドの設定画面でトラッカー選択枠が見づらい
     → プロジェクト設定画面のトラッカー選択タブのように整列させて見やすくしたい。

修正点

  • HTMLの構造およびCSSの調整を実施しました。
  • CSS調整については盛岡さんと相談/確認済み

イメージ

修正前

image

修正後

image

kfischer-okarin and others added 24 commits June 16, 2023 07:22
* add postgres 10.21, 11.16, 12.11, 13.7, 14.3
* use versions existing on hub.docker.com (10.20, 11.15, 12.10, 13.6, 14.2)
* Upgrade ci postgres version from 9.5 to 10 (drop 10.20, 11.15, 12.10, 13.6, 14.2)

Co-authored-by: Ko Nagase <[email protected]>
* Add VSCode devcontainer用の設定を追加
* Add docker-compose.yml
* Move Dockerfile-for-redmine-dev-mirror to .devcontainer/Dockerfile
* Fix Rubyのバージョン変更の手順
* Update README.md
* Update DockerfileでADDしなくなったことによって機能しなくなっていたコードを削除
* Adjust docker-compose.yml MySQL settings and expose PostgreSQL/MySQL ports

* Adjust SQLTools MySQL driver and connections
@telegib telegib requested review from nishidayuya, mk2 and kumojima June 30, 2023 01:56
Copy link

@nishidayuya nishidayuya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです! 🎉 対応おつかれさまでした.

気づいた点は細かいことですのでApproveします.

@@ -1,11 +1,12 @@
<fieldset class="box" id="custom_field_tracker_ids"><legend><%= toggle_checkboxes_link("#custom_field_tracker_ids input[type=checkbox]") %><%=l(:label_tracker_plural)%></legend>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

細かいですが,余分な空行がありそうです.

Suggested change

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます!見落としていました、、、ので、commit を追加しました。

@github-actions
Copy link

Patch can be downloaded here

Copy link
Contributor

@mk2 mk2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants