Skip to content

Conversation

@FabienPapet
Copy link
Contributor

If I change the layout to integrate another one, I have some css overrides in webui.css. This remove bootstrap overrides for navbar

</head>
<body>

<nav class="navbar navbar-light bg-faded">
Copy link
Member

@bocharsky-bw bocharsky-bw May 23, 2018

Choose a reason for hiding this comment

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

Hm, it may be that devs already have configs ID somewhere in their template. I'd rather use CSS class here too like:
<nav class="configs navbar navbar-light bg-faded">

Possible CSS classes duplicating is much better than ID duplicating IMO

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree, will fix this this evening

#configs {
padding: 0 2rem;
}
.navbar-nav .nav-text{
Copy link
Member

Choose a reason for hiding this comment

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

And actually it won't work :) I suppose it should be:
#configs.navbar-nav .nav-text {

i.e. there should be no space between #configs and .navbar-nav since we're talking about the same tag

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Ah, really missed it 👍

width: 6rem;
}

.navbar-nav .nav-link {
Copy link
Member

Choose a reason for hiding this comment

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

And the same here

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

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

Excellent. Thank you

@Nyholm Nyholm merged commit fb70b1d into php-translation:master May 26, 2018
@bocharsky-bw bocharsky-bw mentioned this pull request Jul 4, 2018
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.

3 participants