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

I mentioned to it by mistake. #115

Closed
wants to merge 20 commits into from
Closed

I mentioned to it by mistake. #115

wants to merge 20 commits into from

Conversation

komura-c
Copy link
Owner

@komura-c komura-c commented Sep 4, 2020

No description provided.

@komura-c komura-c requested a review from dninomiya September 4, 2020 04:14
@komura-c komura-c self-assigned this Sep 4, 2020
Comment on lines 91 to 93
useFactory: () =>
environment.production ? {} : { host: 'localhost:8080', ssl: false },
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

ORIGINに合わせて以下の方がシンプルでした

useValue: environment.production ? {} : { host: 'localhost:8080', ssl: false }

Copy link
Owner Author

Choose a reason for hiding this comment

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

レビューとタブスクロールのissueを探していただき、大変ありがとうございます!
試してみます!

@komura-c komura-c marked this pull request as draft September 4, 2020 10:31
@komura-c
Copy link
Owner Author

komura-c commented Sep 4, 2020

ディレクティブ作ってみましたが、あのissueはtabのバグかと思われるんですが、
今回の実装は動的URLでmat-tab-navを使ってるのでrouterを使って考えてみます。
ただTwitterもアプリ版だと戻らないんですが、Web版だと戻るので何もしなくてもいい気もしてきました…
gif
どうしてもちらついてしまいますね…
gifだとちらついてないですね
gif

@dninomiya dninomiya closed this Sep 5, 2020
@komura-c komura-c deleted the feature/112 branch September 5, 2020 03:50
@komura-c komura-c restored the feature/112 branch September 5, 2020 03:51
@komura-c komura-c reopened this Sep 5, 2020
@komura-c komura-c marked this pull request as ready for review September 5, 2020 03:52
@komura-c komura-c closed this Sep 5, 2020
@komura-c komura-c deleted the feature/112 branch September 5, 2020 03:52
dninomiya added a commit that referenced this pull request Sep 5, 2020
@komura-c komura-c changed the title fix #112, #100, #111, #99, #97, #102, 諸修正 I referred to it by mistake. Sep 10, 2020
@komura-c komura-c changed the title I referred to it by mistake. I mentioned to it by mistake. Sep 10, 2020
komura-c pushed a commit that referenced this pull request Nov 3, 2024
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.

2 participants