Revamp screenshot page #7

Merged
MicheleC merged 1 commits from feat/rewamp-screenshot-page into master 1 year ago
Owner

Also add horizontal line before the page footer.

It looks like the attached screenshot.

Also add horizontal line before the page footer. It looks like the attached screenshot.
MicheleC added this to the R14.1.0 release milestone 1 year ago
MicheleC added 1 commit 1 year ago
f1f0b92542
Rewamp screenshot page.
MicheleC force-pushed feat/rewamp-screenshot-page from f1f0b92542 to 10e809b8ee 1 year ago
MicheleC force-pushed feat/rewamp-screenshot-page from 10e809b8ee to 4c4b174f7b 1 year ago
MicheleC changed title from Rewamp screenshot page. to Revamp screenshot page. 1 year ago
MicheleC changed title from Revamp screenshot page. to Revamp screenshot page 1 year ago
MicheleC requested review from SlavekB 1 year ago
MicheleC force-pushed feat/rewamp-screenshot-page from 4c4b174f7b to 8c32587d95 1 year ago
Poster
Owner

Thickness of horizontal line adjusted in last push

Thickness of horizontal line adjusted in last push
SlavekB reviewed 1 year ago
echo " <LINK REL=StyleSheet HREF=\"" . $maindir . "highslide/highslide.css\" TYPE=\"text/css\" MEDIA=\"screen,print\">\n";
}
echo " <meta http-equiv=\"Content-Type\" content=\"text/html;charset=utf-8\">\n";
echo " <style>\nhr.footer {border-top: 0px solid black;}\n</style>\n";
SlavekB commented 1 year ago
Owner

Thank you, the thinner line looks better. However, it makes no sense for this style definition to be here, while all other definitions of styles are in mainstyle.css.

Thank you, the thinner line looks better. However, it makes no sense for this style definition to be here, while all other definitions of styles are in `mainstyle.css`.
Poster
Owner

PR updated :-)

PR updated :-)
MicheleC marked this conversation as resolved
MicheleC force-pushed feat/rewamp-screenshot-page from 8c32587d95 to bc1ed41599 1 year ago
SlavekB approved these changes 1 year ago
SlavekB left a comment
Owner

It looks good.

It looks good.
MicheleC merged commit bc1ed41599 into master 1 year ago
MicheleC deleted branch feat/rewamp-screenshot-page 1 year ago

Reviewers

SlavekB approved these changes 1 year ago
The pull request has been merged as bc1ed41599.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tdemainweb-core#7
Loading…
There is no content yet.