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

Adjust vertical alignment of social media icons #25

Open
bberndt-uaz opened this issue Apr 15, 2022 · 0 comments · May be fixed by #33
Open

Adjust vertical alignment of social media icons #25

bberndt-uaz opened this issue Apr 15, 2022 · 0 comments · May be fixed by #33
Assignees

Comments

@bberndt-uaz
Copy link
Contributor

Problem/Motivation

The social media icons appear slightly misaligned next to their accompanying text.

Describe the bug

Currently, the bottom of the social media icons is not quite aligned with the baseline of the accompanying text (e.g., "Facebook"). See the screenshot below from arizona.edu. Since the icons have a larger height than the text, I believe it would be best to vertically align the center of the icons with the middle of the text.

To Reproduce

Steps to reproduce the behavior:

  1. Enable the Menu Link Attributes module
  2. Add some social media links to the Footer - Social Media menu, adding the necessary classes to the attributes section (e.g., "az-icon-facebook az-icon-spacing" for Facebook)
  3. Look in the footer to confirm that the icons appear misaligned

Proposed resolution

Adjust the CSS for the icons so they are vertically aligned with the middle of the text.

Expected behavior

Icons are vertically aligned with the middle of the text. See screenshot below from arizona.edu.

Screenshots

Current icons on arizona.edu:

image

After adjustment:

image

Desktop (please complete the following information):

  • OS: macOS
  • Browser: Chrome, Safari
  • Version: Chrome 100, Safari 15.4

Smartphone (please complete the following information):

  • Device: Galaxy S9
  • OS: Android 10
  • Browser: Chrome
  • Version: 100
@bberndt-uaz bberndt-uaz added bug Something isn't working low priority labels Apr 15, 2022
@bberndt-uaz bberndt-uaz self-assigned this Apr 15, 2022
@bberndt-uaz bberndt-uaz transferred this issue from az-digital/az_quickstart Apr 22, 2022
@bberndt-uaz bberndt-uaz linked a pull request Jun 3, 2022 that will close this issue
@bberndt-uaz bberndt-uaz removed the bug Something isn't working label Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant