FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix: Troca do menu flutuante. Sai código antigo, entra código novo by ramoncordini · Pull Request #897 · scieloorg/packtools · GitHub

Fix: Troca do menu flutuante. Sai código antigo, entra código novo - #897

Open
ramoncordini wants to merge 1 commit into
scieloorg:masterfrom
ramoncordini:novo-menu-flutuante-artigo
Open

Fix: Troca do menu flutuante. Sai código antigo, entra código novo#897
ramoncordini wants to merge 1 commit into
scieloorg:masterfrom
ramoncordini:novo-menu-flutuante-artigo

Conversation

Copy link
Copy Markdown

O que esse PR faz?

Remove o código antigo do menu flutuante e adiciona o código novo

Onde a revisão poderia começar?

Carregue esse PR e abra o diretorio novo-menu-flutuante. Execute o arquivo artigo-atual.html. Veja o novo menu em funcionamento.

Como este poderia ser testado manualmente?

Siga os passos descritos acima

Algum cenário de contexto que queira dar?

No diretório novo-menu-flutuante/artigo-atual.html existem links de css e js que estão apontados para as ultimas versões de seus arquivos css e js respectivamente. Estes foram colocados apenas para fim de teste. Originalmente os css e js são carregados pelo opac5. Estes são os mesmo arquivos que foram carregados na ultima atualização (scieloorg/opac_5#241)

Screenshots



Quais são tickets relevantes?

Referente ao PR do opac5(scieloorg/opac_5#241)

Referências

Copy link
Copy Markdown
Author

Deve ser removido todo o código com classe floatingMenu e seus respectivos elementos li.

Deve ser adicionado todo o código com classe scielo__floatingMenuCttJs3 e seus respectivos elementos li.

Em caso de dúvidas me comunique.

This file contains hidden or 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
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.

1 participant


Back | FazBrowse Home | New Git URL