Fix acessibilidade rebase #373
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Alterado a viewlet de site actions para retirar o atributo title dos links para adequar as regras de acessibilidade. Alterado a viewlet de serviços para incluir a descrição do objeto no atributo title do link.
ajuste dos links vazios acontent, anavigation e afooter para atender os critérios de acessibilidade
hvelarde
reviewed
Oct 2, 2017
| @@ -6,9 +6,9 @@ | |||
| title view/description"> | |||
| <span id="portal-title-1" | |||
| tal:content="view/title_1">Secretaria de</span> | |||
| <h1 id="portal-title" | |||
| <div id="portal-title" | |||
There was a problem hiding this comment.
IMO, this change could cause problems on third-party themes.
There was a problem hiding this comment.
Unfortunately, this has to be done or we will have accessibility errors in ACCESS tool. We'll add an entry to changelog.
| @@ -0,0 +1,268 @@ | |||
| # -*- coding: utf-8 -*- | |||
| from brasil.gov.portal.browser.viewlets.servicos import ServicosViewlet | |||
|
|
|||
| self.request = self.layer['request'] | ||
| self.adapter = SiteControlPanelAdapter(self.portal) | ||
| self.browser = Browser(self.layer['app']) | ||
| transaction.begin() |
| class PortalLogoTestCase(AccessibilityTestCase): | ||
|
|
||
| def portal_logo_tema_test(self, cor): | ||
| """ Testa se o portal logo está presente em todos os temas |
| class AcessibilidadeViewletTestCase(AccessibilityTestCase): | ||
|
|
||
| def test_viewlet_is_present(self): | ||
| """ Testa se a viewlet foi registrada corretamente |
| ) | ||
| api.content.create( | ||
| type='Link', | ||
| container=self.servicos, | ||
| id='servico-1', | ||
| title=u'Servico 1' | ||
| title=u'Servico 1', | ||
| remoteUrl=u'http://www.google.com' |
There was a problem hiding this comment.
you're missing a semicolon at the end.
| ) | ||
| api.content.create( | ||
| type='Link', | ||
| container=self.servicos, | ||
| id='servico-2', | ||
| title=u'Servico 2' | ||
| title=u'Servico 2', | ||
| remoteUrl=u'http://www.plone.org' |
There was a problem hiding this comment.
you're missing a semicolon at the end.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Boas práticas para o PR: Checklist
Foi aberta uma issue relativa a esse PR;
Foram adicionados testes unitários;
Foram adicionados testes robots;
Foram adicionados upgradeSteps;
Foi adicionada a modificação no CHANGES.rst do pacote, sempre no topo do arquivo (ou seja, na primeira linha do release ainda não lançado), contendo o seu nome de usuário do github e a referência ao issue que você está tratando (não esqueça de colocar no fim do arquivo a url para a issue, use o padrão do próprio arquivo);
PR não contém assuntos diferentes. PR's devem ter um objetivo claro para facilitar o review. Ex: não junte, num mesmo PR, alterações de code-analysis e de implementação. Faça um PR de code-analysis e depois o da implementação em si;
Evite muitos commits pequenos de um mesmo assunto no PR, sempre que possível efetue rebase;
Com a aprovação de plonegovbr/brasil.gov.temas#140 e o lançamento de um release de brasil.gov.temas poderemos remover
@unittest.expectedFailurenum outro PR para lançar um release de brasil.gov.portal.