search

up2university / docker-moodle

A Dockerfile that installs the latest Moodle, Apache, PHP, and MySQL
3 stars 9 forks source link

Add hyperlinks on the contact section #108

Closed NelsonDiasFCCN closed 5 years ago

NelsonDiasFCCN commented 6 years ago

We need to create hyperlinks on the bottom part of the learning platform in the contact section:

screenshot 2018-11-30 at 14 38 29

msbachler commented 5 years ago

What the Fordson theme config seems to do is rather than links is the footer it adds the orange icons, like the globe one that is already showing in the image above, below the textual contact string. So the orange globe icon is already a link to the website. It does those 'links' with the theme settings under the 'Social Icons' section on the theme Footer page. Those are its 'links'. So following that way of doing things I know how to add another icon link for the contact with an envelope icon that would look like this this:

108

Is that what you want? If it is, I can then figure out how to persist that in the theme envs. Or do you want to extend the theme to do something else like turn the text of the url and contact into a links directly?

msbachler commented 5 years ago

Or we could just have the icons and remove the test bit like this:

108b

NelsonDiasFCCN commented 5 years ago

I prefer the last example with the icons.

1 - Can we increase the size of the icons and align them vertically? 2 - Can we add a description to each button “Website” and “Contact us” at the right side of each button?

Best regards, Nelson

On 17 Dec 2018, at 15:04, Michelle S Bachler notifications@github.com wrote:

Or we could just have the icons and remove the test bit like this:

https://user-images.githubusercontent.com/4311824/50095276-16b06480-020d-11e9-8c7a-15041f4306a5.PNG — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/up2university/docker-moodle/issues/108#issuecomment-447876614, or mute the thread https://github.com/notifications/unsubscribe-auth/AQCy7seOpqQEuH_DmnsePJqN7Gg-vQrJks5u57KZgaJpZM4Y71N7.

Nelson Dias Área de Serviços Avançados | Serviço Técnico de Vídeo

Fundação para Ciência e a Tecnologia Unidade FCCN - Computação Científica Nacional Av. do Brasil, 101, 1700-066 Lisboa, Portugal T: [+351] 218 440 100 nelson.dias@fccn.pt | www.fccn.pt

msbachler commented 5 years ago

We can't change the icon size or add text, at least not with properties in the theme Footer settings page.

msbachler commented 5 years ago

I think you can add custom css though which might at least let me make the icons bigger. I will investigate.

msbachler commented 5 years ago

OK. I can overide the class '.socialicon' by changing the theme_fordson | scss setting and set the font-size to whatever you like (the icons are actually a font element). The font used for the icons is 14px at present. Here is what 24px looks like as an example:

108c

msbachler commented 5 years ago

I can possibly align them vertically by overriding the css some more. But I can't add text beside them with css changes, so I am not sure aligning them vertically makes sense without the text next to them?

NelsonDiasFCCN commented 5 years ago

Yes, it doesn’t make sense to align vertically if we can’t include the text. It seems much better now.

Thanks, Nelson

On 17 Dec 2018, at 16:12, Michelle S Bachler notifications@github.com wrote:

I can possibly align them vertically by overriding the css some more. But I can't add text beside them with tcss changes, so I am not sure aligning them vertically makes sense without the text next to them?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/up2university/docker-moodle/issues/108#issuecomment-447901301, or mute the thread https://github.com/notifications/unsubscribe-auth/AQCy7iJMhspj64hxrT3RcXkmOMlF4gZKks5u58KIgaJpZM4Y71N7.

Nelson Dias Área de Serviços Avançados | Serviço Técnico de Vídeo

Fundação para Ciência e a Tecnologia Unidade FCCN - Computação Científica Nacional Av. do Brasil, 101, 1700-066 Lisboa, Portugal T: [+351] 218 440 100 nelson.dias@fccn.pt | www.fccn.pt

msbachler commented 5 years ago

OK. I will go ahead and implement these changes then.

NelsonDiasFCCN commented 5 years ago

Thanks

Nelson

On 17 Dec 2018, at 16:41, Nelson Dias nelson.dias@fccn.pt wrote:

Yes, it doesn’t make sense to align vertically if we can’t include the text. It seems much better now.

Thanks, Nelson

On 17 Dec 2018, at 16:12, Michelle S Bachler <notifications@github.com mailto:notifications@github.com> wrote:

I can possibly align them vertically by overriding the css some more. But I can't add text beside them with tcss changes, so I am not sure aligning them vertically makes sense without the text next to them?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/up2university/docker-moodle/issues/108#issuecomment-447901301, or mute the thread https://github.com/notifications/unsubscribe-auth/AQCy7iJMhspj64hxrT3RcXkmOMlF4gZKks5u58KIgaJpZM4Y71N7.

Nelson Dias Área de Serviços Avançados | Serviço Técnico de Vídeo

Fundação para Ciência e a Tecnologia Unidade FCCN - Computação Científica Nacional Av. do Brasil, 101, 1700-066 Lisboa, Portugal T: [+351] 218 440 100 nelson.dias@fccn.pt mailto:nelson.dias@fccn.pt | www.fccn.pt http://www.fccn.pt/

Nelson Dias Área de Serviços Avançados | Serviço Técnico de Vídeo

Fundação para Ciência e a Tecnologia Unidade FCCN - Computação Científica Nacional Av. do Brasil, 101, 1700-066 Lisboa, Portugal T: [+351] 218 440 100 nelson.dias@fccn.pt | www.fccn.pt