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

Change tabs appearance #9512

Closed
wants to merge 3 commits into from

Conversation

atm-john
Copy link
Member

1 - Default
2 - Hover
3 - Active
4 - Improve badge visibility
new tab css

@atm-john atm-john mentioned this pull request Sep 20, 2018
@eldy
Copy link
Member

eldy commented Sep 20, 2018

Is look and badge ok on smartphone too ?
On smartphone, the tabs are merged into a combo list.

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Sep 20, 2018
@atm-john
Copy link
Member Author

Good game ! I forgot the mobile testing part. Now it's fixed !
I added a note for me for the next time.

}
a.tab:hover
{
/*
background: rgba(<?php echo $colorbacktabcard1; ?>, 0.5) url(<?php echo dol_buildpath($path.'/theme/'.$theme.'/img/nav-overlay3.png',1); ?>) 50% 0 repeat-x;
color: #<?php echo $colortextbacktab; ?>;
*/
text-decoration: underline;
text-decoration: none;
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an accessibility error! Please keep in mind accessibility rules.
See https://www.sitepoint.com/15-rules-making-accessible-links/ (chapter 11) and https://www.w3.org/TR/WCAG20-TECHS/F73.html

@atm-john
Copy link
Member Author

This is represented by top border of the tab.
I added borders to make tabs more "button like" ( for exemple .butAction have text-decoration )

I'm agree with this type of recommandations for links in text paragraph, but we are not on a text paragraph and it's not the link's normal state.

The hover state change his border top line color saturation, hue and change cursor to pointer.

If hover state style need more visibility for accessibility, I suggest changing hover state by the same of active state (remove bottom border and change top border to a 2px colored style. But actually I don't think it's necessary, don't you ?

@librethic-code
Copy link
Contributor

You're right we are not into textual context. This point related to Dolibarr interface generally…

For tabs IMHO we need more contrast between active tabs and others (don't forget :focus state, using same styles than :hover), not only one border. Cursor changes is not valid when user navigate with keyboard with Tab key.

In any case, we applaud you for this courageous work on Dolibarr theming ;)

@atm-john
Copy link
Member Author

atm-john commented Oct 2, 2018

Added : more hover visual effect

@eldy
Copy link
Member

eldy commented Oct 4, 2018

Having borders on each tab gives a very loaded screen.
We retreive the look of tab we had in old version of Dolibarr before we removed the borders on tab to follow what bootstrap suggests (see screenshot). Also when we have several tabs on 2 lines (for example on small screen on tablet, we don't see anymore where is the badge (it is at both at top of a title but at bottom of another one. this is minor, this does not happen too frequently). So can you just add a constant so we can keep the "non selected tab" as simple links for people who want a cleaner interface ?

image

@atm-john
Copy link
Member Author

atm-john commented Oct 5, 2018

yeah i see.

@atm-john atm-john closed this Oct 23, 2018
@atm-john atm-john deleted the appearance_change_tab branch June 4, 2021 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants