Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix for dark theme #26

Merged
merged 7 commits into from
Apr 26, 2016
Merged

Hotfix for dark theme #26

merged 7 commits into from
Apr 26, 2016

Conversation

dojineko
Copy link
Contributor

@dojineko dojineko commented Apr 25, 2016

FIXIT

  • Markdown Previewのテーブルスタイル
  • メモ削除時モーダルのボタンカラー
  • AceEditorの下部 border-radius

Markdown Previewのテーブルスタイル

image

ダークテーマを追加した時にMarkdownプレビュー時の
テーブルのスタイルのオーバーライドが漏れてましたので追加でスタイルを定義しました。

メモ削除時モーダルのボタンカラー

image

メモ削除時のモーダル表示のボタンスタイルが未定義だったため追加しました。

AceEditorの下部 border-radius

下記変更にともなって生じた、AceEditor切替時に出る白い余白を修正しました。
934e4d9#diff-9facd39aca37bb6f900638a5ae61410aR386

その他微調整

  • code タグの背景色を設定
  • リストのフォルダ名に文字色を設定し位置調整

image

@dojineko
Copy link
Contributor Author

@Rokt33r
お手数おかけして申し訳ないですがレビューお願いします 🙇

@Rokt33r
Copy link
Member

Rokt33r commented Apr 25, 2016

あ、自分も気づけませんでした。今週中に反映させます!

@dojineko dojineko changed the title Apply dark theme to markdown table Hotfix for dark theme Apr 25, 2016
@dojineko
Copy link
Contributor Author

dojineko commented Apr 25, 2016

他にもないか自分で見て回ってみます 🙇
メモの削除時に出るモーダルの「No」ボタンのHoverカラーとかも設定したほうが
良さそうなので修正予定です!

@Rokt33r Rokt33r mentioned this pull request Apr 25, 2016
@dojineko
Copy link
Contributor Author

@Rokt33r
気づいたところを修正しました。お手すきにチェックお願いします 🙇

@Rokt33r
Copy link
Member

Rokt33r commented Apr 25, 2016

お疲れ様でした!確認してから水曜日の午前中にリリースし直します! 😃

@Rokt33r Rokt33r merged commit a0c83f3 into BoostIO:master Apr 26, 2016
@Rokt33r
Copy link
Member

Rokt33r commented Apr 26, 2016

問題ないようです。明日0時ごろにDeployします!

@dojineko dojineko deleted the hotfix-dark branch April 26, 2016 05:55
@dojineko
Copy link
Contributor Author

ご確認ありがとうございました!
よろしくお願いします 🙇

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

Successfully merging this pull request may close these issues.

None yet

2 participants