Skip to content

Commit

Permalink
Moved to kiwix.css some CSS with global effect
Browse files Browse the repository at this point in the history
Moved from index.css into kiwix.css some CSS with global effect thus
making it apply to the viewer too.

Extra font-size directives in taskbar.css are needed to undo the effect
of 'font-size: 62.5%' now applied to the 'html' element type.
  • Loading branch information
veloman-yunkan committed Oct 28, 2023
1 parent 3fa25b8 commit 036a2cb
Show file tree
Hide file tree
Showing 5 changed files with 34 additions and 32 deletions.
22 changes: 0 additions & 22 deletions static/skin/index.css
Original file line number Diff line number Diff line change
@@ -1,25 +1,3 @@
*,
*::after,
*::before {
margin: 0;
padding: 0;
box-sizing: inherit;
}

html {
font-size: 62.5%;
}

body {
position: relative;
box-sizing: border-box;
}

::selection {
background-color: #00b4e4;
color: white;
}

.kiwixNav {
background-color: #f4f6f8;
width: 100%;
Expand Down
22 changes: 22 additions & 0 deletions static/skin/kiwix.css
Original file line number Diff line number Diff line change
@@ -1,3 +1,25 @@
*,
*::after,
*::before {
margin: 0;
padding: 0;
box-sizing: inherit;
}

html {
font-size: 62.5%;
}

body {
position: relative;
box-sizing: border-box;
}

::selection {
background-color: #00b4e4;
color: white;
}

.modal-wrapper {
position: fixed;
z-index: 100;
Expand Down
4 changes: 3 additions & 1 deletion static/skin/taskbar.css
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
}

.kiwixsearch {
font-size: 1.6rem;
position: relative;
height: 26px;
width: 100%;
Expand Down Expand Up @@ -119,6 +120,7 @@ label[for=kiwixsearchbox] {
}

a.suggest, a.suggest:visited, a.suggest:hover, a.suggest:active {
font-size: 1.6rem;
display: block;
text-decoration: none;
color: inherit;
Expand All @@ -142,7 +144,7 @@ a.suggest, a.suggest:visited, a.suggest:hover, a.suggest:active {
}

#uiLanguageSelector .modal-content #ui_language {
font-size: 1rem;
font-size: 1.6rem;
width: 100%;
}

Expand Down
4 changes: 2 additions & 2 deletions test/library_server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1028,12 +1028,12 @@ TEST_F(LibraryServerTest, no_name_mapper_catalog_v2_individual_entry_access)
" <title>Welcome to Kiwix Server</title>\n" \
" <link\n" \
" type=\"text/css\"\n" \
" href=\"/ROOT%23%3F/skin/kiwix.css?cacheid=28889647\"\n" \
" href=\"/ROOT%23%3F/skin/kiwix.css?cacheid=bdae4d22\"\n" \
" rel=\"Stylesheet\"\n" \
" />\n" \
" <link\n" \
" type=\"text/css\"\n" \
" href=\"/ROOT%23%3F/skin/index.css?cacheid=624104e6\"\n" \
" href=\"/ROOT%23%3F/skin/index.css?cacheid=081e1606\"\n" \
" rel=\"Stylesheet\"\n" \
" />\n" \
" <link rel=\"apple-touch-icon\" sizes=\"180x180\" href=\"/ROOT%23%3F/skin/favicon/apple-touch-icon.png?cacheid=f86f8df3\">\n" \
Expand Down
14 changes: 7 additions & 7 deletions test/server.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ const ResourceCollection resources200Compressible{
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/i18n.js" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/i18n.js?cacheid=6a8c6fb2" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/index.css" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/index.css?cacheid=624104e6" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/index.css?cacheid=081e1606" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/index.js" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/index.js?cacheid=ce19da2a" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/iso6391To3.js" },
Expand All @@ -71,7 +71,7 @@ const ResourceCollection resources200Compressible{
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/mustache.min.js" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/mustache.min.js?cacheid=bd23c4fb" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/taskbar.css" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/taskbar.css?cacheid=786a623c" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/taskbar.css?cacheid=04cb5531" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/viewer.js" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/viewer.js?cacheid=201653b8" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/fonts/Poppins.ttf" },
Expand Down Expand Up @@ -141,7 +141,7 @@ const ResourceCollection resources200Uncompressible{
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/hash.png" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/hash.png?cacheid=f836e872" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/kiwix.css" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/kiwix.css?cacheid=28889647" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/kiwix.css?cacheid=bdae4d22" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/magnet.png" },
{ STATIC_CONTENT, "/ROOT%23%3F/skin/magnet.png?cacheid=73b6bddf" },
{ DYNAMIC_CONTENT, "/ROOT%23%3F/skin/search-icon.svg" },
Expand Down Expand Up @@ -276,8 +276,8 @@ TEST_F(ServerTest, CacheIdsOfStaticResources)
const std::vector<UrlAndExpectedResult> testData{
{
/* url */ "/ROOT%23%3F/",
R"EXPECTEDRESULT( href="/ROOT%23%3F/skin/kiwix.css?cacheid=28889647"
href="/ROOT%23%3F/skin/index.css?cacheid=624104e6"
R"EXPECTEDRESULT( href="/ROOT%23%3F/skin/kiwix.css?cacheid=bdae4d22"
href="/ROOT%23%3F/skin/index.css?cacheid=081e1606"
<link rel="apple-touch-icon" sizes="180x180" href="/ROOT%23%3F/skin/favicon/apple-touch-icon.png?cacheid=f86f8df3">
<link rel="icon" type="image/png" sizes="32x32" href="/ROOT%23%3F/skin/favicon/favicon-32x32.png?cacheid=79ded625">
<link rel="icon" type="image/png" sizes="16x16" href="/ROOT%23%3F/skin/favicon/favicon-16x16.png?cacheid=a986fedc">
Expand Down Expand Up @@ -315,8 +315,8 @@ R"EXPECTEDRESULT( <img src="${root}/skin/download
},
{
/* url */ "/ROOT%23%3F/viewer",
R"EXPECTEDRESULT( <link type="text/css" href="./skin/kiwix.css?cacheid=28889647" rel="Stylesheet" />
<link type="text/css" href="./skin/taskbar.css?cacheid=786a623c" rel="Stylesheet" />
R"EXPECTEDRESULT( <link type="text/css" href="./skin/kiwix.css?cacheid=bdae4d22" rel="Stylesheet" />
<link type="text/css" href="./skin/taskbar.css?cacheid=04cb5531" rel="Stylesheet" />
<link type="text/css" href="./skin/css/autoComplete.css?cacheid=08951e06" rel="Stylesheet" />
<script type="module" src="./skin/i18n.js?cacheid=6a8c6fb2" defer></script>
<script type="text/javascript" src="./skin/languages.js?cacheid=8d132878" defer></script>
Expand Down

0 comments on commit 036a2cb

Please sign in to comment.