Opened 5 years ago

Last modified 5 years ago

#2669 closed change

Wrap header, footer and nav content with corresponding tags — at Version 3

Reported by: saroyanm Assignee:
Priority: P4 Milestone:
Module: User-Interface Keywords:
Cc: fhd, greiner Blocked By: #1524
Blocking: #2407 Platform: Unknown
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by saroyanm)


With #1524 we are going to redesign Adblock Plus options page.
With #1526 we have implemented skeleton and initial logic, but we don't wrap any content with HTML5 header, footer and nav tags

What to change

Wrap the corresponding content of the new option page with the header, footer and nav tags:

  • change the top left div in sidebar with the id "page-title" to header tag.
  • wrap with <nav> tag main navigation elements (id="main-navigation-tabs") that contains General, Advanced and Help tab navigation.
  • wrap with <footer> tag the navigation on the bottom of sidebar ("Share" and "Donate").

Change History (3)

comment:1 Changed 5 years ago by greiner

  • Blocking 2407 added
  • Priority changed from Unknown to P4

Could you be a bit more specific about which elements should be put into (or turned into) which HTML5 element? It's not obvious from the current description.

comment:2 Changed 5 years ago by greiner

  • Cc greiner added

comment:3 Changed 5 years ago by saroyanm

  • Description modified (diff)
Note: See TracTickets for help on using tickets.