Project

General

Profile

Actions

Bug #8389

closed

Add style when show_navigation_menu is enabled

Added by Angel Quiroz over 6 years ago. Updated over 6 years ago.

Status:
Bug resolved
Priority:
High
Category:
Styles, Layout & Accessibility
Target version:
Start date:
08/08/2016
Due date:
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Files

1.9.x.png (248 KB) 1.9.x.png Angel Quiroz, 08/08/2016 17:20
1.10.png (106 KB) 1.10.png Angel Quiroz, 08/08/2016 17:20
asas.png (119 KB) asas.png Alex Aragon, 16/08/2016 23:33
shortcut.png (29.2 KB) shortcut.png Yannick Warnier, 17/08/2016 04:45
ejemplo.png (79.8 KB) ejemplo.png Alex Aragon, 17/08/2016 22:48
Actions #1

Updated by Angel Quiroz over 6 years ago

  • Enable show_navigation_menu
  • Check (any) course home
Actions #2

Updated by Yannick Warnier over 6 years ago

  • Subject changed from Add stlye when show_navigation_menu is enabled to Add style when show_navigation_menu is enabled
Actions #3

Updated by Alex Aragon over 6 years ago

Ahora el diseño esta asi...

Actions #4

Updated by Alex Aragon over 6 years ago

  • Status changed from New to Needs testing
  • Assignee changed from Alex Aragon to Yannick Warnier

verificar si se muestra en todos los cursos y si esta correcto el efecto que le puse.

Actions #5

Updated by Yannick Warnier over 6 years ago

  • File shortcut.png shortcut.png added
  • Status changed from Needs testing to Assigned
  • Assignee changed from Yannick Warnier to Alex Aragon
  • % Done changed from 0 to 30
Está bien por lo general peeeeeero:
  • el texto no está bien alineado (tiene que estar unos 5 pixeles más a bajo en comparación con el icono)
  • el icono debería estar del lado derecho y aparecer aunque esté "escondida" la lenguita, porque sino es un poco difícil ubicar donde se encuentra la herramienta correcta

De otro lado, ahora ya no aparecen los atajos a herramientas (la barra horizontal con los iconos que sí aparecen en la 1.10)

Actions #6

Updated by Alex Aragon over 6 years ago

Modificaciones.

Actions #8

Updated by Alex Aragon over 6 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from Alex Aragon to Yannick Warnier
  • % Done changed from 30 to 70
Actions #9

Updated by Yannick Warnier over 6 years ago

  • Status changed from Needs testing to Assigned
  • Assignee changed from Yannick Warnier to Alex Aragon
  • % Done changed from 70 to 80

After checking, I just asked Alex to review it so the items of the side menu do not move when you first load the page.

Actions #10

Updated by Yannick Warnier over 6 years ago

  • % Done changed from 80 to 70
Still missing:
  • shortcut icons only appear on course homepage
  • navigation_menu should not move when a new page is loaded
  • when show_navigation_menu is on "icons only", icons appear in the middle of the page
Actions #11

Updated by Yannick Warnier over 6 years ago

  • Priority changed from Normal to High
Actions #13

Updated by Yannick Warnier over 6 years ago

  • % Done changed from 70 to 80

Ya casi. Solo falta el problema de la lista de iconos verticales en el medio de la página.

Actions #14

Updated by Alex Aragon over 6 years ago

  • Assignee changed from Alex Aragon to Yannick Warnier
Actions #15

Updated by Yannick Warnier over 6 years ago

  • Status changed from Assigned to Bug resolved
  • % Done changed from 80 to 100

Aprobado.

Actions

Also available in: Atom PDF