Ольга

можно рамкой

.button::before{
  content:'';
  display:inline-block;
  border-right:10px solid red;
  border-top:10px solid transparent;
  height:0;
  width:0;
}

можно картинкой(фоновой или псевдоэлемент, если мутно - заюзать svg),
можно через новое свойство css clip-path https://webformyself.com/znakomstvo-s-css-svojstvo...

Для начала вы бы определились,а как оно будет выглядеть на мобильной версии, как будет выглядеть при переполнении(несколько записей в одной ячейке), а потом уже определяться с версткой.
ИМХО(да и не только мое), верстать нужно всегда с мобильной версии, с оглядкой на десктопную,а не наоборот. И дело не только в принципе Mobile First от Google, но и в том,что в мобильном варианте элементы идут в порядке потока информации,а на декстопе уже перемешиваются(перескакивают вправо-лево). если делать в обратном порядке, придется идти более сложным путем, т.к. придется менять элементы по вертикали, если изначальный порядок не был правильно угадан.

Если предугадать высоту ячеек нельзя, то лучший способ-это таблицы. Да и в целом - он самый простой. На мобильной версии либо добавляется скролл, либо творится непотребство в виде задания ячейкам и строкам display:block (вполне ходовой вариант,не стоит бояться). Основная проблема таблиц: в ячейках не совсем одинаково может работать position:absolute.

Нужно сбросить и назначить оба класса,как-то так:

$(document).on('click', 'a', function () {
    if ($(this).is(".icon-arrow-up")) {
      let line=$(this).closest('.MenuTableRow');
      let tbody=$(this).closest('tbody');

      line.prependTo(tbody);
      
      let rows=$('.MenuTableRow',tbody);
      rows.removeClass('row-first').removeClass('row-last');
      rows.first().addClass('row-first');
      rows.last().addClass('row-last');
    }
    if ($(this).is(".icon-arrow-down")) {
      let line=$(this).closest('.MenuTableRow');
      let tbody=$(this).closest('tbody');
      
      line.appendTo('tbody');
      
      let rows=$('.MenuTableRow',tbody);
      rows.removeClass('row-first').removeClass('row-last');
      rows.first().addClass('row-first');
      rows.last().addClass('row-last');
    }
});

Для изменения жирности нужно подгрузить шрифт с этой жирностью. Браузер сам, конечно, может некоторым образом симулировать жирность или курсив, но если шрифт не системный для каждой жирности, начертания и их сочетаний нужно подгружать свои файлы. При этом их нужно еще корректно объявлять, например:

@font-face {
	font-family: 'Commissioner';
	src: url('fonts/Commissioner/Commissioner-Regular.eot');
	src: local('Commissioner Regular'), local('Commissioner-Regular'),
		url('fonts/Commissioner/Commissioner-Regular.eot?#iefix') format('embedded-opentype'),
		url('fonts/Commissioner/Commissioner-Regular.woff2') format('woff2'),
		url('fonts/Commissioner/Commissioner-Regular.woff') format('woff'),
		url('fonts/Commissioner/Commissioner-Regular.ttf') format('truetype');
	font-weight: normal;
	font-style: normal;
}
@font-face {
	font-family: 'Commissioner';
	src: url('fonts/Commissioner/Commissioner-Medium.eot');
	src: local('Commissioner Medium'), local('Commissioner-Medium'),
		url('fonts/Commissioner/Commissioner-Medium.eot?#iefix') format('embedded-opentype'),
		url('fonts/Commissioner/Commissioner-Medium.woff2') format('woff2'),
		url('fonts/Commissioner/Commissioner-Medium.woff') format('woff'),
		url('fonts/Commissioner/Commissioner-Medium.ttf') format('truetype');
	font-weight: 500;
	font-style: normal;
}

имя у шрифта одно, а вот жирность разная. При этом лучше не забывать о разных браузерах а также о том,что шрифты могут быть в системе пользователя(в этом случае ничего грузиться не будет). В целом, формат .eot - это атавизм для IE, но до сих пор лежит в готовых паках до кучи.

Тут скорее вопрос личного выбора и того,как изначально разрабатывался сайт. теги типа article и figure относятся к семантической верстке стандарта html5 , подробнее можно почитать тут https://htmlacademy.ru/blog/boost/frontend/semantics. Если сайт уже придерживается стратегии использования семантики - этому нужно продолжать следовать.

тут сразу несколько ошибок:

1. по $arResult['MORE_PHOTO'] вы проходитесь при помощи foreach а в итоге переменными не пользуетесь ($key,$img), $arResult[PROPERTIES][MORE_PHOTO][VALUE][0] - всегда берете нулевой элемент,результат очевиден, так что читаем про циклы и учимся ими пользоваться
   
2. судя по тому,что хоть что-то отображается,нужно проходиться не по $arResult['MORE_PHOTO'], а по $arResult['PROPERTIES']['MORE_PHOTO']['VALUE']
   
3. $arResult[PROPERTIES][MORE_PHOTO][VALUE][0] - у ключей отсутствуют кавычки, но это строки,а не коснстанты, хоть интерпретатор сожрет - так делать нехорошо.
   

Сбрось стили кнопки:

border:none;
background:none;

Как вариант создать обработчик,который будет регистрироваться в момент открытия модального окна (либо вешать на событие открытия,либо разместить в самой модалке).
в итоге скрипт будет выглядеть как-то так:

let elements = document.querySelectorAll('.modal-body li a');//делаем выборку строк,на которых нужна обработка клика
elements.onclick = function() { //вешаем обработчик
    let id=this.id; //забираем id
    let inp=document.querySelectorAll('.form-group [name="parent_id"]'); //находим инпут
    inp.value=id; //втавляем id  в значение инпута
};

есть пара замечаний по верстке:

• использовать тег <a> не для ссылок - плохая практика, теги нужно использовать по назначению
  
• для верстки и скриптов лучше использовать классы либо id, привязка тегам в обоих случаях ведет к проблемам в будущем.
  
• для хранения данных(в данном случае id) лучше использовать атрибуты data, которые были придуманы как раз для этого
  

задай блоку отрицательный margin-top
или картинку опусти