Ticket #871 (closed enhancement: fixed)

Opened 9 months ago

Last modified 4 months ago

inline style의 제거

Reported by: graphittie Owned by: graphittie
Priority: minor Milestone: 1.7.4
Component: Skin Version: 1.7
Keywords: skin,style,inline style,css Cc:
Release:

Description

PHP에 의해 HTML element 출력시 inline style을 출력하는 부분을 최대한 제거한다. inline style은:

  • 스킨이 DOCTYPE strict 모드(1.0 strict 또는 1.1)를 사용할 경우 validation에서 에러로 체크된다.
  • structure와 presentation을 뒤섞어 유지보수에 시간을 소비하게 한다.

제거된 style은 style/system.css로 보아 하위 호환성을 유지하도록 하되, 이 파일은 스킨의 하위호환성이 버려질 때(2.0?) deprecated 될 것이다.

Change History

  Changed 9 months ago by graphittie

  • status changed from new to assigned

  Changed 9 months ago by graphittie

[5573]

  • style/system.css 파일 추가. 하위 호환성 유지를 위해, 제거된 inline style을 모아두는 파일.

  Changed 9 months ago by graphittie

[5574]

  • PHP에 의해 출력되는 404 에러 메세지의 inline style 제거. TCwarning이라는 클래스 명을 부여하여 하위 호환성을 유지.

  Changed 9 months ago by graphittie

[5575]

  • #TTSearchForm의 inline style 제거.

  Changed 9 months ago by graphittie

[5576]

  • 에디터 첨부파일 목록인 #fileList와 관련된 element의 inline style 제거.
  • #fileList를 #TCfilelist로 개명.
  • .enclosure 추가.

  Changed 9 months ago by graphittie

[5577]

  • [5574]의 나머지 완전 제거.
  • style/system.css 파일 출력을 위한 코드 추가.

  Changed 9 months ago by graphittie

[5578]

  • 에디터의 서식 목록 부분의 inline style 제거.

  Changed 9 months ago by graphittie

[5579]

  • 글 메뉴의 명령 콤보박스의 기본 option inline style 제거.

  Changed 9 months ago by graphittie

[5580]

  Changed 9 months ago by graphittie

[5581]

follow-ups: ↓ 12 ↓ 13   Changed 9 months ago by graphittie

TO DO

XPath.class.php: 삽질 심하게 해야 함.;;

in reply to: ↑ 11   Changed 9 months ago by inureyes

Replying to graphittie:

==== TO DO ==== XPath.class.php: 삽질 심하게 해야 함.;;

XPath의 경우에는 외부 라이브러리이므로 그 쪽 버전이 오르면 덮어 씌울 부분입니다. 삽질 심하게 안하셔도...

in reply to: ↑ 11 ; follow-up: ↓ 15   Changed 9 months ago by coolengineer

Replying to graphittie:

==== TO DO ==== XPath.class.php: 삽질 심하게 해야 함.;;

그 파일 내용을 보니 디버그용이던데, 디버그에 관련된 정보도 하시려고요?

  Changed 9 months ago by graphittie

[5584]

  • 작업 도중 발생한 매치되지 않는 괄호 제거.

in reply to: ↑ 13   Changed 9 months ago by graphittie

Replying to coolengineer:

Replying to graphittie:

==== TO DO ==== XPath.class.php: 삽질 심하게 해야 함.;;

그 파일 내용을 보니 디버그용이던데, 디버그에 관련된 정보도 하시려고요?

헉. 그런가요; 검색 때리니 주르륵 나오길래;;

  Changed 8 months ago by inureyes

[5597]

  • [5576] 관련 첨부파일의 본문 삽입이 불가능해지는 버그 수정

  Changed 8 months ago by inureyes

  • status changed from assigned to closed
  • resolution set to fixed

  Changed 8 months ago by inureyes

  • version changed from 1.6 to 1.7

  Changed 4 months ago by inureyes

  • status changed from closed to reopened
  • resolution deleted
  • milestone changed from 1.7 to 1.8

  Changed 4 months ago by inureyes

Changeset: [6442]

#871

  Changed 4 months ago by inureyes

Changeset: [6443]

#871

  • [6442] 의 내용을 전부 blog 출력의 begin쪽으로 통합

  Changed 4 months ago by inureyes

Changeset: [6447]

#871

  Changed 4 months ago by inureyes

Changeset: [6457]

#871

  • [6447] 에서 경로가 1.8 기준인 부분을 수정

  Changed 4 months ago by inureyes

[6501]

  • [6457] 에서 경로가 1.8 이상 기준으로 된 부분 수정
  • 버전번호 변경

  Changed 4 months ago by inureyes

  • status changed from reopened to closed
  • resolution set to fixed
  • milestone changed from 1.8 to 1.7.3.1
Note: See TracTickets for help on using tickets.