Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Anpassung CSS/SASS Code für Icons #98

Open
xwolfde opened this issue Nov 12, 2024 · 2 comments
Open

Anpassung CSS/SASS Code für Icons #98

xwolfde opened this issue Nov 12, 2024 · 2 comments
Labels
Design Issues zum Thema Gestaltung

Comments

@xwolfde
Copy link
Member

xwolfde commented Nov 12, 2024

Es soll keine Icon-Font verwendet werden, sondern die Ausgaben mittels CSS-Background SVG gesetzt werden:
Hierzu können die entsprechenden SASS-Anweisungen aus dem FAU EInrichtungen Theme fast 1:1 übernommen werden; Diese verfügen bereits über einen umfangreichen Satz an Icons für fast alle Social Media- und Forschungsportale:

Anmerkung:
Bitte nicht schlicht das FAU Einrichtungen Theme oder dessen Pfade verlinken/referenzieren, sondern den Code tatsächlich übernehmen. Das Plugin soll ja auch auf andere Themes nutzbar sein.

@xwolfde xwolfde added the Design Issues zum Thema Gestaltung label Nov 12, 2024
@xwolfde
Copy link
Member Author

xwolfde commented Dec 3, 2024

Es würde mir reichen, wenn ihr es so macht, dass überall dort, wo die Social Media Icons erscheinen sollen, diese in dieser Form gemacht werden:

<ul class="socialmedia">
    <li> <span class="screen-reader-text"> <?php echo esc_html(ucfirst($icon_data['name'])); ?>: </span><a href="<?php echo esc_url($social['url']); ?>"><?php echo esc_url($social['url']); ?></a></li>
   ..

</ul>

Diese class="socialmedia" ist bereits vom Theme vordefiniert. Zwar ist das dann nicht bei anderen Themes enthalten, aber da kann ich mich noch drum kümmern.
Dann braucht ihr darin nur ganz normale Links zu den Plattformen setzen. Das jeweilige Icon wird dann ausgehend von der URL automatisch gestellt. Ihr braucht daher keinerlei extra Definition machen. Das kann ich dann im Finetuning II machen unter #104

Im Falle von page.php würde die Änderung ab Z124 so aussehen:

 <?php if (in_array('socialmedia', $show_fields) && !in_array('socialmedia', $hide_fields)): ?>
                                    <?php if (!empty($person['contacts'][0]['socials'])) : ?>
                                       <span class="screen-reader-text"><?php echo esc_html__('Social Profiles:', 'rrze-faudir'); ?></span>
                                        <ul class="socialmedia">
                                            <?php foreach ($person['contacts'][0]['socials'] as $social):
                                                $icon_data = get_social_icon_data($social['platform']);
                                            ?>
                                               <li><span class="screen-reader-text"><?php echo esc_html(ucfirst($icon_data['name'])); ?>: </span><a href="<?php echo esc_url($social['url']); ?>" itemprop="sameAs" ><?php echo esc_url($social['url']); ?></a></li>
                                            <?php endforeach; ?>
                                        </div>
                                    <?php endif; ?>
                                <?php endif; ?>

(Die Attribute target="_blank" und rel="noopener noreferrer" werfen wir in dem Zusammenhang weg. Wir nutzen kein target bei Links.)

@xwolfde
Copy link
Member Author

xwolfde commented Dec 6, 2024

Die Template wurden angepasst. Daher schieb ich diesen Issue nun in meinen Milestone 2.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design Issues zum Thema Gestaltung
Projects
None yet
Development

No branches or pull requests

2 participants