Fix problem of PL #956 #962

Closed

mskf1383 wants to merge 3 commits from (deleted):main into main

mskf1383 commented

2021-11-29 13:44:21 +00:00

Contributor

The pull request #956 has problem with posts with long title. In posts with long title, description went behind the title.
So I made some other changes to fix that:

.card header {     /*******/
    display: flex; /* New */
}                  /*******/
.card h3 {
    flex-grow: 1;
    margin: 0;
    font-family: "Vazir","Playfair Display",serif;
    font-size: 1.75em;
    font-weight: normal;
    line-height: 1.75;
    display: inline-block;
    position: absolute; /* removed */
}

I hope there isn't any more problems!

The pull request https://git.joinplu.me/Plume/Plume/pulls/956 has problem with posts with long title. In posts with long title, description went behind the title. So I made some other changes to fix that: ```css .card header { /*******/ display: flex; /* New */ } /*******/ .card h3 { flex-grow: 1; margin: 0; font-family: "Vazir","Playfair Display",serif; font-size: 1.75em; font-weight: normal; line-height: 1.75; display: inline-block; position: absolute; /* removed */ } ``` I hope there isn't any more problems!

mskf1383 added 2 commits 2021-11-29 13:44:25 +00:00

Update 'assets/themes/default/_global.scss' 4d0d412ad3

Update 'assets/themes/default/_global.scss' 04284e0ca5

mareklach commented

2021-12-01 17:16:26 +00:00

Contributor

@mskf1383 Please don't remove position: absolute; completely, I changed it to position: relative; in #960 in order to make what you're adressing work on mobile devices like the iPhone too!

Thanks,

@mskf1383 Please don't remove `position: absolute;` completely, I changed it to `position: relative;` in #960 in order to make what you're adressing work on mobile devices like the iPhone too! Thanks,

mskf1383 commented

2021-12-02 09:32:48 +00:00

Author

Contributor

@mskf1383 Please don't remove position: absolute; completely, I changed it to position: relative; in #960 in order to make what you're adressing work on mobile devices like the iPhone too!

Thanks,

I don't think this make problem for mobile screens.
I attach some pictures.

My first change:

Your change:

This change:

> @mskf1383 Please don't remove `position: absolute;` completely, I changed it to `position: relative;` in #960 in order to make what you're adressing work on mobile devices like the iPhone too! > > Thanks, I don't think this make problem for mobile screens. I attach some pictures. My first change: ![My first change](https://git.joinplu.me/attachments/d7e2732e-402f-4b19-9823-c567e5a621cc) Your change: ![Your change](https://git.joinplu.me/attachments/89ff18ed-f918-4d12-80e8-6b922d8cb86e) This change: ![This change](https://git.joinplu.me/attachments/09254ab9-7b60-43b4-b68a-3fcbc3aefb6d)

نماگرفت از 2021-12-02 12-52-32.png

42 KiB

نماگرفت از 2021-12-02 12-53-32.png

43 KiB

نماگرفت از 2021-12-02 12-52-58.png

42 KiB

mskf1383 commented

2021-12-02 09:37:43 +00:00

Author

Contributor

And nothing happen if I add a position: relative to it

And nothing happen if I add a `position: relative` to it