[GH-ISSUE #305] compose:markdown icons missing #265

Closed
opened 2026-02-25 21:34:36 +03:00 by kerem · 4 comments
Owner

Originally created by @zeigerpuppy on GitHub (Nov 26, 2018).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/305

Originally assigned to: @jasonmunro on GitHub.

Issue

With the current version of cypht, I am having difficulty seeing the icons is the compose toolbar for Markdown.

screenshot from 2018-11-26 20-31-14

The web server logs show this error

[:error]  PHP Notice:  Uninitialized string offset: 0 in /mnt/cypht/modules/core/message_functions.php on line 269, referer: http://cypht.server.net/?page=compose
  • There are no errors in the browser console log (Debian 9 Firefox).
  • The site is served locally for testing at the moment via a very simple apache config.
  • Memcached is enabled and settings are in flat files at the moment.
<Virtualhost *:80>
ServerName cypht.server.net

    DocumentRoot /var/www/cypht
    <Directory />
        Options FollowSymLinks
        AllowOverride None
    </Directory>

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

</Virtualhost>
Originally created by @zeigerpuppy on GitHub (Nov 26, 2018). Original GitHub issue: https://github.com/cypht-org/cypht/issues/305 Originally assigned to: @jasonmunro on GitHub. ## Issue With the current version of cypht, I am having difficulty seeing the icons is the compose toolbar for Markdown. ![screenshot from 2018-11-26 20-31-14](https://user-images.githubusercontent.com/415981/49005242-58279400-f1ba-11e8-8ef9-ab648dda4e05.png) The web server logs show this error ``` [:error] PHP Notice: Uninitialized string offset: 0 in /mnt/cypht/modules/core/message_functions.php on line 269, referer: http://cypht.server.net/?page=compose ``` - There are no errors in the browser console log (Debian 9 Firefox). - The site is served locally for testing at the moment via a very simple apache config. - Memcached is enabled and settings are in flat files at the moment. ``` <Virtualhost *:80> ServerName cypht.server.net DocumentRoot /var/www/cypht <Directory /> Options FollowSymLinks AllowOverride None </Directory> ErrorLog ${APACHE_LOG_DIR}/error.log CustomLog ${APACHE_LOG_DIR}/access.log combined </Virtualhost> ```
kerem 2026-02-25 21:34:36 +03:00
Author
Owner

@jasonmunro commented on GitHub (Nov 27, 2018):

@zeigerpuppy the warning is an unrelated issue that I will fix shortly. I can confirm that the issue is that we are forcing a global font in the theme CSS which is breaking the font based icons in the markdown editor. I will update here when I have a fix. Thanks for the feedback!

<!-- gh-comment-id:442143266 --> @jasonmunro commented on GitHub (Nov 27, 2018): @zeigerpuppy the warning is an unrelated issue that I will fix shortly. I can confirm that the issue is that we are forcing a global font in the theme CSS which is breaking the font based icons in the markdown editor. I will update here when I have a fix. Thanks for the feedback!
Author
Owner

@jasonmunro commented on GitHub (Nov 27, 2018):

@zeigerpuppy should be fixed in the master branch. It's a css change so you will need to rerun the config gen script to recreate the production site.css file.

<!-- gh-comment-id:442179858 --> @jasonmunro commented on GitHub (Nov 27, 2018): @zeigerpuppy should be fixed in the master branch. It's a css change so you will need to rerun the config gen script to recreate the production site.css file.
Author
Owner

@zeigerpuppy commented on GitHub (Nov 29, 2018):

Thanks! The fix is tested and working well. You've done a great job with the markdown implementation, it's simple and inter-operates well with other mail clients.

<!-- gh-comment-id:442692624 --> @zeigerpuppy commented on GitHub (Nov 29, 2018): Thanks! The fix is tested and working well. You've done a great job with the markdown implementation, it's simple and inter-operates well with other mail clients.
Author
Owner

@jasonmunro commented on GitHub (Nov 29, 2018):

Thanks for the follow up and for checking out the markdown feature - not sure anybody but myself has tried it! :)

<!-- gh-comment-id:442925807 --> @jasonmunro commented on GitHub (Nov 29, 2018): Thanks for the follow up and for checking out the markdown feature - not sure anybody but myself has tried it! :)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cypht#265
No description provided.