Skip to content

tabs was created#1

Open
OlgaTitarenko wants to merge 2 commits into
masterfrom
dev
Open

tabs was created#1
OlgaTitarenko wants to merge 2 commits into
masterfrom
dev

Conversation

@OlgaTitarenko
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown

@vadim-ilchenko vadim-ilchenko left a comment

Choose a reason for hiding this comment

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

Просто и понятно. Принимаю. Оставил комментарии про <Fragment> - иногда он очень сильно выручает.
p.s. Там в задании было еще добавить console.log(title) при изменении активной табы. Посмотри, пожалуйста.

Comment thread src/App.js
class App extends Component {
render() {
return (
<div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно использовать <Fragment> для оберток, если тебе надо вернуть контент, не оборачивая его в еще один контейнер.
https://reactjs.org/docs/fragments.html

Comment thread src/Tab.js

function Tab (props) {
return (
<div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тут тоже не очень понятно, зачем оборачивать в div. Хотя для стилизации в перспективе это ОК

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